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 2023-05-31 #19896

Closed
37 tasks done
OSBotify opened this issue May 31, 2023 · 21 comments
Closed
37 tasks done

Deploy Checklist: New Expensify 2023-05-31 #19896

OSBotify opened this issue May 31, 2023 · 21 comments
Assignees
Labels
🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging StagingDeployCash

Comments

@OSBotify
Copy link
Contributor

OSBotify commented May 31, 2023

Release Version: 1.3.21-2
Compare Changes: production...staging

This release contains changes from the following pull requests:

Internal QA:

Deploy Blockers:

Deployer verifications:

  • I checked the App Timing Dashboard and verified this release does not cause a noticeable performance regression.
  • I checked Firebase Crashlytics and verified that this release does not introduce any new crashes. More detailed instructions on this verification can be found here.
  • I checked GitHub Status and verified there is no reported incident with Actions.

cc @Expensify/applauseleads

@Julesssss Julesssss self-assigned this May 31, 2023
@AndrewGable AndrewGable added the 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging label May 31, 2023
@OSBotify
Copy link
Contributor Author

🚀 All staging deploys are complete, @Expensify/applauseleads please begin QA on version https://github.com/Expensify/App/releases/tag/1.3.21-2 🚀

@AndrewGable AndrewGable self-assigned this May 31, 2023
@AndrewGable
Copy link
Contributor

Expensify is going to be leading the testing of this checklist as we have a deadline to get this out by tonight.

@AndrewGable
Copy link
Contributor

#19528

@isagoico
Copy link

Will post regression updates here as well. As mentioned in this thread we will prioritize the following platforms for the regression test plan:

  1. Web
  2. iOS
  3. Android
    Once we get those platforms out of the way, we'll proceed with Web and Desktop.

@AndrewGable let us know if there's any PRs that we should test. We're dividing efforts in the Applause team and we could assign 1 or 2 people to help with PRs.

@mvtglobally
Copy link

#19618 checking off as team already validated

@AndrewGable
Copy link
Contributor

@isagoico @mvtglobally - Can Applause start from the bottom and just leave a comment on which PRs you are working on? e.g. start with #19868

@dangrous
Copy link
Contributor

checked off #19412 - some minor unexpected results but nothing to prevent deploy

@JmillsExpensify
Copy link

Heads up #19621 doesn't pass though it's pretty minor. The entire page scrolls on mWeb/Safari when you try to scroll within the suggestions pop-up for emojis and mentions. I wouldn't call this a blocker though, shall I go ahead and sign off?

@AndrewGable
Copy link
Contributor

Yep, let's make an comment on the issue, create a follow up issue and then check it off.

@JmillsExpensify
Copy link

Cool. I'll comment on the PR now and check off.

@trjExpensify
Copy link
Contributor

QA'd #19618, tests pass. 👍

@isagoico
Copy link

@isagoico @mvtglobally - Can Applause start from the bottom and just leave a comment on which PRs you are working on? e.g. start with #19868

Yep! On it. Grabbing #19868

@JmillsExpensify
Copy link

#19738 passed on all platforms

@JmillsExpensify
Copy link

JmillsExpensify commented May 31, 2023

#19732 this flow and QA can no longer be tested now that passwordless is everywhere. I also confirmed here. Checking off.

@AndrewGable
Copy link
Contributor

Checking off #19733 per @alexpensify's comment here: #19733 (comment)

@isagoico
Copy link

#19868 Results:

  1. Web - ✅
  2. mWeb - ✅
  3. iOS - ✅
  4. Android - ✅
  5. Desktop - ✅

Pass in all platforms 🎉 Checking it off.

@trjExpensify
Copy link
Contributor

Hitting some failures on Android and mWeb Safari executing the Concierge QA steps on #19724 here.

@marcaaron
Copy link
Contributor

Checking off #19919 as it's NotABlocker

@AndrewGable
Copy link
Contributor

Checked off #19724 with a comment for a follow up here: #19724 (comment)

@AndrewGable
Copy link
Contributor

Decided against a deploy blocker for #19923 with this information here: #19923 (comment)

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

10 participants