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 2023-08-03 #24121

Closed
86 tasks done
OSBotify opened this issue Aug 3, 2023 · 22 comments
Closed
86 tasks done

Deploy Checklist: New Expensify 2023-08-03 #24121

OSBotify opened this issue Aug 3, 2023 · 22 comments
Assignees
Labels
🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging StagingDeployCash

Comments

@OSBotify
Copy link
Contributor

OSBotify commented Aug 3, 2023

Release Version: 1.3.50-0
Compare Changes: production...staging

This release contains changes from the following pull requests:

Deploy Blockers:

Deployer verifications:

  • I checked the App Timing Dashboard and verified this release does not cause a noticeable performance regression.
  • I checked Firebase Crashlytics and verified that this release does not introduce any new crashes. More detailed instructions on this verification can be found here.
  • I checked GitHub Status and verified there is no reported incident with Actions.

cc @Expensify/applauseleads

@Beamanator
Copy link
Contributor

Oof we've got a lot of old PRs again 🙃 we can make a new section and check them off like last time (I'll help with that tomorrow morning)

@mvtglobally mvtglobally added the 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging label Aug 3, 2023
@OSBotify
Copy link
Contributor Author

OSBotify commented Aug 3, 2023

🚀 All staging deploys are complete, @Expensify/applauseleads please begin QA on version https://github.com/Expensify/App/releases/tag/1.3.50-0 🚀

@mvtglobally
Copy link

Checking off as previously tested
#22467
#22912
#23152
#23462
#23531
#23641
#23704
#23707
#23715
#23754
#23772
#23773
#23777
#23780
#23843
#23844
#23845
#23854
#23857
#23860
#23878
#23879
#23907
#23912
#23915
#23917
#23927
#23975
#23980
#23983
#23988
#24003
#24010
#24025
#24036
#24109

@Beamanator
Copy link
Contributor

Checking off #24123 since we marked in NAB

@Beamanator
Copy link
Contributor

Checking off #24131 b/c it was fixed & CP'd to staging 👍

@Beamanator
Copy link
Contributor

Checking off #24160 since it's related to "receipt scanning" which is under beta - but it still should be fixed soon

@mvtglobally
Copy link

Can you help with #23648 (comment)

@Beamanator
Copy link
Contributor

Yes @mvtglobally - i tested #23648 and checked it off b/c it's working well! 👍

@mvtglobally
Copy link

@Beamanator We are down to 2% TR left

Current summary
#23527 pending validation on all platforms
#23621 pending validation on all platforms
#23636 is failing with #23436
We are blocked #23648 (comment)
#23828 pending validation on all platforms
#23934 is failing #24147
#23938 is failing #24158
#23947 pending validation on all platforms
#23965 pending validation on all platforms
#23971 pending validation on all platforms
#23972 pending validation on all platforms
#23977 pending validation on all platforms

@Beamanator
Copy link
Contributor

Thanks for the update @mvtglobally ! I have to head out now for the weekend, I posted in internal channels, mentioning your update 👍 So someone else mayyy take this over this afternoon, but most likely we'll pick this back up on Monday 👍

@Beamanator
Copy link
Contributor

Last note for the day:

@mvtglobally
Copy link

@Beamanator We are down to 1% TR left

Current summary
#23636 is failing with #23436
#23828 pending validation on all platforms
#23934 is failing #24147
#23938 is failing #24158
#23971 pending validation on all platforms
#23977 pending validation on all platforms

@Beamanator
Copy link
Contributor

Checking #24124 off since its fix is on staging 👍

@Beamanator
Copy link
Contributor

Checking off #24193 as we considered it a dupe

Checking off #24147 b/c we CPd its fix (#24203) to staging & it tested well, according to @MonilBhavsar

@puneetlath
Copy link
Contributor

Checking off #24183 as it's only occurring on dev so isn't a deploy blocker.

@puneetlath
Copy link
Contributor

Checking off #23934 based on this comment.

@puneetlath
Copy link
Contributor

puneetlath commented Aug 7, 2023

Checking off #23938 since it was already there on production, so even if it isn't working, it's not a deploy blocker. I've asked the issue assignees to re-test.

@puneetlath
Copy link
Contributor

Checking off #23636 since it's not a deploy blocker. Commented in the PR so that they can address the fix not working.

@puneetlath
Copy link
Contributor

Checking off #23971. I tested it on web and iOS staging and it seems to be working as expected.

@puneetlath
Copy link
Contributor

Checking off #23977. I tested on iOS and it seems to be working.

@puneetlath
Copy link
Contributor

:shipit:

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 StagingDeployCash
Projects
None yet
Development

No branches or pull requests

5 participants