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-10-04 #5636

Closed
13 tasks done
OSBotify opened this issue Oct 4, 2021 · 11 comments
Closed
13 tasks done

Deploy Checklist: New Expensify 2021-10-04 #5636

OSBotify opened this issue Oct 4, 2021 · 11 comments
Assignees
Labels
🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging StagingDeployCash

Comments

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

isagoico commented Oct 4, 2021

Starting QA!

@isagoico
Copy link

isagoico commented Oct 5, 2021

@flodnv
Copy link
Contributor

flodnv commented Oct 5, 2021

  1. Ah! This should be solved when https://github.com/Expensify/Web-Secure/pull/2060 gets deployed to production which should be within a few hours!

@isagoico
Copy link

isagoico commented Oct 5, 2021

We had a big doubt with this issue #5655 on if it was expected behaviour or not. I think that there should not be an option to pay when requesting money. We added the deploy blocker label since this is not reproducible in production. CC @nickmurray47 (tagging you since you worked on #3866)

@nickmurray47
Copy link
Contributor

Taking a look - that is not the expected behavior.

@nickmurray47
Copy link
Contributor

I have a PR #5661 in review that should fix the issue once deployed to Staging

@isagoico
Copy link

isagoico commented Oct 5, 2021

Checking off #5661 (comment) and #5655

@isagoico
Copy link

isagoico commented Oct 5, 2021

@isagoico
Copy link

isagoico commented Oct 6, 2021

Update on #5615 (comment). It's currently failing since it's throwing the wrong error when entering a really old age.

@roryabraham
Copy link
Contributor

roryabraham commented Oct 6, 2021

Demoting #5667 from deploy blocker and checked it off the list because it's an extreme edge case that we don't need to address with urgency. Also checked off #5615 because we don't need to block deploy on this.

@chiragsalian I think you should be good to :shipit: whenever you're ready 🚀

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

7 participants