Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deploy Checklist: New Expensify 2021-09-15 #5279

Closed
27 tasks done
OSBotify opened this issue Sep 15, 2021 · 18 comments
Closed
27 tasks done

Deploy Checklist: New Expensify 2021-09-15 #5279

OSBotify opened this issue Sep 15, 2021 · 18 comments
Assignees
Labels
🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging Monthly KSv2 StagingDeployCash

Comments

@OSBotify
Copy link
Contributor

OSBotify commented Sep 15, 2021

Release Version: 1.0.99-4
Compare Changes: production...staging

This release contains changes from the following pull requests:

Deploy Blockers:

cc @Expensify/applauseleads

@mvtglobally mvtglobally added the 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging label Sep 15, 2021
@mvtglobally
Copy link

starting QA

@roryabraham
Copy link
Contributor

Checking off #5229 because we were able to verify that it works as expected.

@Julesssss
Copy link
Contributor

Checking off #5291 as it only occurs on one platform and doesn't block the user from sharing PDF files.

@francoisl
Copy link
Contributor

I removed the deploy blocker label from #5286 because it's also happening in production, doesn't look like a new bug.

@Julesssss
Copy link
Contributor

In order to resolve #5290, I have raised a revert PR here. Once this is merged and on the deploy Checklist, I will mark #5290 off the list.

@isagoico
Copy link

Checking off #5216 (comment)

@isagoico
Copy link

Checking off #5307 (comment) and #5305 since they were tested and it was a pass 🎉

@Julesssss
Copy link
Contributor

#5290 has been resolved as it was already occuring.

@roryabraham
Copy link
Contributor

@marcaaron Can you confirm if we are good to check #5330 off on the checklist? Judging by this comment it looks like we are good there?

@marcaaron
Copy link
Contributor

Yep tested fine for me.

@roryabraham
Copy link
Contributor

Is this checklist blocked on the verification code bug for switching between E.com and NewDot?

@isagoico
Copy link

Yes, we were unable to complete QA for #5244

@MelvinBot MelvinBot added the Monthly KSv2 label Sep 20, 2021
@bondydaa
Copy link
Contributor

checking off #5244 because we reverted it on #5360 so we could unblock this checklist

@Jag96
Copy link
Contributor

Jag96 commented Sep 20, 2021

#5244 has been reverted in #5360, CPing to staging. Once its on staging I'll check it off the list and move forward w/ the deploy.

@Jag96
Copy link
Contributor

Jag96 commented Sep 20, 2021

Staging deploy was a success, checking off #5360. Looks like there aren't any blockers left here so going to close and proceed w/ the prod deploy!

:shipit:

@Jag96 Jag96 closed this as completed Sep 20, 2021
@OSBotify
Copy link
Contributor Author

This issue either has unchecked QA steps or has not yet been marked with the :shipit: emoji of approval.
Reopening!

@OSBotify OSBotify reopened this Sep 20, 2021
@Jag96
Copy link
Contributor

Jag96 commented Sep 20, 2021

:shipit:

@Jag96 Jag96 closed this as completed Sep 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging Monthly KSv2 StagingDeployCash
Projects
None yet
Development

No branches or pull requests