-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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-24 #25813
Comments
The following PRs should have been
These PRs are for ExpensifyHelp (already in production, production QA), checking them off: |
🚀 All staging deploys are complete, @Expensify/applauseleads please begin QA on version https://github.com/Expensify/App/releases/tag/1.3.57-0 🚀 |
#25860 is NAB, checking it off |
Checking #25830 off, fixed in staging |
#25797 was fixed on staging for the last checklist, I just forgot to remove the deploy blocker label. Checking it off |
#25871 is behind a beta so NAB, checking it off |
#25879 is NAB, checking it off |
@roryabraham Can you check this one |
Also, is it possible to QA #24535 internally? |
Checking off #25890 as I don't think we need to be blocking deploy on it. |
#25841 is fixed, checking it off |
Demoting #25874, checking it off |
@roryabraham We are down to last 4% TR and few PRs Current summary #24535 - Can you QA internally? #25161 blocked on Web/mWeb due to #25696. & Issue found on Native apps #25871 |
@mvtglobally three CPs listed above: #25813 (comment) |
@roryabraham for some reason earlier messages didnt show till now. Got the CPs. working on those |
Checking off #25827 as internal review passed 👍 |
Regression is completed. #24535 is pass on all but IOS. We are not able to test on IOS as links are not opening in the app but in mWeb. checking off Current summary - unchecked PRs |
Demoted #25884, checking it off |
Checking off #25889. Demoted to NAB since it's a super small UI issue and only happens in DEV. |
Using this legend: ❗ No status 🏁 #25859, should be fixed by #25882 in 1.3.57-3 |
🆗 #25888 demoted |
Checking #25966 off, demoted as its a new design |
#25878 is fixed in staging, checking off |
Going to check off #22622 because all the associated regressions are fixed or otherwise resolved |
Marking #24034 as a pass. Looks good |
#25161 is behind a beta, checking it off |
#25434 is behind a beta, checking it off |
Ok, that leaves us with just two blockers:
I've self-assigned both of these, so I'm going to triage them now |
Demoting #25893, checking it off. |
#25979 is not reproducible on staging, checking it off |
Demoted #26074 and checked it off |
Checking off #26027. Fix merged and being CPed. I'll test once it hits staging. |
Tested #26055 in staging. It's a pass. Checking off. |
Down to #25921 |
Checked off #25921 |
Checklist is clear. Shipping it. |
Release Version:
1.3.57-5
Compare Changes: production...staging
This release contains changes from the following pull requests:
Internal QA:
Deploy Blockers:
Deployer verifications:
cc @Expensify/applauseleads
The text was updated successfully, but these errors were encountered: