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-12-08 #6645

Closed
25 tasks done
OSBotify opened this issue Dec 8, 2021 · 16 comments
Closed
25 tasks done

Deploy Checklist: New Expensify 2021-12-08 #6645

OSBotify opened this issue Dec 8, 2021 · 16 comments
Assignees
Labels
🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging StagingDeployCash

Comments

@MelvinBot
Copy link

📣 @applausebot You have been assigned to this job by @OSBotify!
Please apply to this job in Upwork and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑‍💻
Keep in mind: Code of Conduct | Contributing 📖

@roryabraham
Copy link
Contributor

Nice short checklist here 😄

@isagoico
Copy link

isagoico commented Dec 8, 2021

My favorites ❤️

@roryabraham
Copy link
Contributor

@isagoico Feel free to lock the checklist and begin QA as soon as you're ready.

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

Starting QA

@roryabraham
Copy link
Contributor

The fix for #6667 was deployed in 1.1.19-1, and the fix for #6666 was deployed in 1.1.19-2, so feel free to retest those whenever you're ready.

@roryabraham
Copy link
Contributor

Manually CPing #6684 to staging in 1.1.19-4 to fix #6675 and also to test #6655.

@roryabraham
Copy link
Contributor

1.1.19-4 is deploying here. Once that's done we should be able to retest all existing deploy blockers.

@mvtglobally
Copy link

Regression is almost complete. Few re-tests in progress

Deploy blockers found (we are re-testing)

Comments added
#6608 (comment)

Issues logged
#6676
#6681
#6685
#6688
#6691
#6689
#6669

@roryabraham
Copy link
Contributor

Thanks @mvtglobally, keep me posted with the status of deploy blocker retests.

@roryabraham
Copy link
Contributor

Any update here? Seems like we should be almost ready-to-go here.

@mvtglobally
Copy link

mvtglobally commented Dec 10, 2021

@roryabraham, Regression is almost done.
Blockers are re-tested and passed. I checked off all PRs and issues #6666 , #6667

We had a bit inconsistent notifications behavior on Android. Team is trying to re-verify "Receive notification while app is backgrounded" steps. We should be able to close this in the next hour.

@mvtglobally
Copy link

@roryabraham We are able to repro this #5022 today. I dont think its a deploy blocker as this is still open.

Regarding #6675 We are getting very mixed results.
Few testers still able to repro on build 1.1.19-4. I am not able to repro on any device. Can you also double check?

https://user-images.githubusercontent.com/43995119/145620704-05838316-adc0-4841-a5fa-bedf768b3f8e.mp4
https://user-images.githubusercontent.com/43995119/145620894-bf3ad384-c2d1-450f-850e-f13b86ab0444.mp4

@roryabraham
Copy link
Contributor

roryabraham commented Dec 10, 2021

Shoot, I was able to reproduce on #6675 iOS 1.1.19-4 😞 Going to reopen and look into it some more.

@roryabraham
Copy link
Contributor

Going to demote #6675 from deploy blocker to KI. @mvtglobally Please include it in your weekly retests.

@roryabraham
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

6 participants