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: Expensify.cash 2021-05-14 #2948

Closed
26 tasks done
OSBotify opened this issue May 14, 2021 · 10 comments
Closed
26 tasks done

Deploy Checklist: Expensify.cash 2021-05-14 #2948

OSBotify opened this issue May 14, 2021 · 10 comments
Assignees
Labels
🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging StagingDeployCash

Comments

@OSBotify
Copy link
Contributor

OSBotify commented May 14, 2021

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

This release contains changes from the following pull requests:

Deploy Blockers:

cc @Expensify/applauseleads

@isagoico
Copy link

Starting QA 🚀

@isagoico isagoico added the 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging label May 17, 2021
@isagoico
Copy link

#2843 was a fail, being fixed here #2971

@isagoico
Copy link

isagoico commented May 17, 2021

Regression is finished!
Issues found:

  1. Android - App crashes when uploading a PDF file #2972 - Possible deploy blocker
  2. Edit message - Able to save an empty message #2968
  3. [HOLD for payment 23 June] Web - Chat - Adjust modal rendering  #2967

PRs are finished too!
No comments added.

Thanks!

@isagoico isagoico removed the 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging label May 18, 2021
@roryabraham
Copy link
Contributor

#2843 was fixed in #2971

@roryabraham
Copy link
Contributor

Okay, due to an incident with the GH servers, this checklist was not being correctly updated. So I went ahead and manually added the following PRs to the checklist:

All these PRs were merged since this checklist was closed yesterday 😬

@AndrewGable it would be awesome if the checklist-building didn't work by adding 1 PR at a time and instead looked at the commit history to find all the PRs that were merged in the timeframe we're interested in. That way, this would be less brittle when GH has an incident.

@roryabraham
Copy link
Contributor

Created a follow-up issue here to improve how this checklist is populated in the future.

@isagoico
Copy link

Starting QA 🚀

@isagoico isagoico added the 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging label May 18, 2021
@isagoico
Copy link

@roryabraham
Copy link
Contributor

Looks like the only issues we have are related to IOU stuff. Since IOU's are still hidden behind a beta, those issues should not be treated as deploy blockers. So I'm thinking we'll run a production deploy 🚀

@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

4 participants