-
Notifications
You must be signed in to change notification settings - Fork 3k
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-12-29 #6920
Comments
Starting QA |
@nickmurray47 The InternalQA PR #4329 will not be available to QA until Web-E PR https://github.com/Expensify/Web-Expensify/pull/32181 will be deployed, right? |
Ah yes, that's correct @mountiny! |
@yuwenmemon @roryabraham We are still waiting for IOS and Desktop builds. Can you please check |
I think this may be a regression: #6940 I haven't been able to test if this problem is in production because I don't have bank credentials to test there. |
Removed |
@nickmurray47 Therefore I have check the PR off. We will QA it together with the Web-E PR and all the features are under beta so no need to worry about that. |
Checking off #6947. It has been fixed |
Checking off #6948. Re-test is a pass |
Checking off #6932 . Re-test is a pass |
@francoisl Are we good to check off #6811 since #6931 is no longer a blocker? |
@mvtglobally Yes looks like it. Also, I see that there are other issues that are still marked as deploy blockers above, but had their blocker label removed since they were added, namely:
And then, #6932 is confirmed to be fixed, so it would mean all deploy blockers are cleared and we can deploy? |
Checking off |
all blockers have been resolved. |
Internal QA done for #4329 |
Going to rerun this, hoping that it fixes https://expensify.slack.com/archives/C03V9A4TB/p1641234691374100 |
Sorry for the multiple reopens, we're trying to fix various issues with the deploy today. That last one mostly worked, but there was an issue with creating a new deploy checklist because the release version in the original message of this issue was wrong. I'm going to change it to |
Release Version:
1.1.24-8
Compare Changes: production...staging
This release contains changes from the following pull requests:
Update Address search - show all inputs #6569
Anthony remove second secure inputs #6809
Changed the Amount Text to TextInput #6811
Corrected - IOU Cursor is set to pointer in IOU preview #6832
Rename ExpensifyText #6835
Show permission error dialog only when storage permission is denied or missing #6840
Clean up ReimbursementAccountUtils #6843
validate value in "Name on card" field #6845
Refactor BankAccountSetup using nextStepValues to force Chase manual account numbers #6851
Initial Setup for wallet Transfer #6852
Increase upload size for avatar image #6864
fix overlapping text #6880
Hide Keyboard shortcuts menu in Native Devices #6881
Add onfido logs #6892
[NO QA] Minor refactor - polish #6897
fix: New chat aren't showing bold in Left Hand Nav #6898
Fix IOU Page #6909
Internal QA:
Deploy Blockers:
cc @Expensify/applauseleads
The text was updated successfully, but these errors were encountered: