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-11-22 #31667

Closed
62 tasks done
github-actions bot opened this issue Nov 22, 2023 · 28 comments
Closed
62 tasks done

Deploy Checklist: New Expensify 2023-11-22 #31667

github-actions bot opened this issue Nov 22, 2023 · 28 comments
Assignees
Labels
Daily KSv2 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging StagingDeployCash

Comments

@github-actions
Copy link
Contributor

github-actions bot commented Nov 22, 2023

Release Version: 1.4.2-3
Compare Changes: production...staging

This release contains changes from the following pull requests:

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

Copy link

melvin-bot bot commented Nov 22, 2023

📣 @github-actions[bot]! 📣
Please report bugs or suggest features in the #expensify-open-source Slack channel, don't directly open issues in this repo!
Instructions here to join the channel 📖

@melvin-bot melvin-bot bot closed this as completed Nov 22, 2023
@os-botify
Copy link

os-botify bot commented Nov 22, 2023

Sorry, only members of @Expensify/Mobile-Deployers can close deploy checklists.
Reopening!

@os-botify os-botify bot reopened this Nov 22, 2023

This comment was marked as off-topic.

@kavimuru kavimuru added the 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging label Nov 22, 2023
@OSBotify
Copy link
Contributor

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

@mountiny
Copy link
Contributor

Btw, made a PR to fix this issue with the checklist being closed upon creation https://github.com/Expensify/Web-Expensify/pull/39805

@kavimuru
Copy link

kavimuru commented Nov 22, 2023

Regression is 24% progress

#31517 is blocked in Desktop and intended as per this comment #25790 (comment) and checking it off.
#31545 failing with #31707

@Julesssss
Copy link
Contributor

Julesssss commented Nov 22, 2023

@luacmartins blocker #31688 is about to be resolved here, but I might be out for the day by then. Would you mind keeping an eye on the fix and CPing it?

@luacmartins
Copy link
Contributor

Checking off #31688. Demoted to NAB since it's a navigation wide issue that affects App on reloading a page and it's not that sensitive.

@luacmartins
Copy link
Contributor

Checking off #31704. Demoted to NAB.

@luacmartins
Copy link
Contributor

Checking off #31725 and #31734. We reverted the offending PR

@luacmartins
Copy link
Contributor

Checking off #31690. We reverted the offending PR

@kavimuru
Copy link

kavimuru commented Nov 22, 2023

Regression is 81% progress.
#31287 failing with 31688 and 31592 (KI)
#31294 failing with 31592 (KI)- ok to check it off?
#31517 and #31527 are blocked in Desktop and intended as per this comment #25790 (comment) and checking it off.
#31545 failing with #31707

@marcaaron
Copy link
Contributor

Checking off #31745

@luacmartins
Copy link
Contributor

Checking off #31749 and #31751. We reverted the offending PR and it's no longer reproducible in staging.

@luacmartins
Copy link
Contributor

Checking off #31755. We reverted the PR that introduced this new functionality so the steps won't work.

@luacmartins
Copy link
Contributor

Checking off #31752. Users can already do this, the PR is question is only adding the required copy to the input

@luacmartins
Copy link
Contributor

@mvtglobally checking off #27670, since we reverted the PR

@luacmartins
Copy link
Contributor

luacmartins commented Nov 22, 2023

Checking off #31738 and #31743, tested in staging and related blockers are no longer reproducible

@luacmartins
Copy link
Contributor

Checking off #31287. We merged a fix.

@luacmartins
Copy link
Contributor

Checking off #31294. The type error no longer happens, but we hit another KI.

@luacmartins
Copy link
Contributor

Checking off #31545. This is intended for now since we reverted the PR that introduced the skeleton, but just merged the PR that reintroduces it today.

@kavimuru
Copy link

Regression is just 3 steps left

#28453 failing with #31757

@luacmartins
Copy link
Contributor

Checked off #31760. Demoted to NAB

@kavimuru
Copy link

Regression is completed.

Unchecked PR:

#28453 failing with #31757

@luacmartins
Copy link
Contributor

Checking off #31763. We CPed a fix.

@luacmartins
Copy link
Contributor

Checking off #28453, since the feature is still WIP and it doesn't break any functionality.

@luacmartins luacmartins self-assigned this Nov 23, 2023
@luacmartins luacmartins added the Daily KSv2 label Nov 23, 2023
@luacmartins
Copy link
Contributor

Checklist is clear. Kicking off the deploy.

@luacmartins
Copy link
Contributor

:shipit:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Daily KSv2 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging StagingDeployCash
Projects
None yet
Development

No branches or pull requests

7 participants