Skip to content

[Due for payment 2025-04-03] [MEDIUM] Use Mobile-Expensify staging branch for NewDot staging builds #55403

Closed

Description

@mountiny

Problem

Every staging E/App build uses Mobile-Expensify main. This means that the following scenario is possible (and has caused fires 🔥!):

  1. Staging QA is near completion in E/App
  2. A pull request is merged in Mobile-Expensify that breaks something in HybridApp
  3. An unrelated E/App PR is CP’d to staging. When this happens, the PR from the previous step is included in the new staging build
  4. Because QA was already near completion, we ship the checklist without noticing that the PR from step 2 broke something. Now production is broken 💥

Solution

  1. Create staging and production branches in Mobile-Expensify (following the same branch protections as E/App
  2. Update the staging and production branches in Mobile-Expensify at the same cadences as NewDot. In other words, when we close the checklist and do a production deploy, main becomes staging and staging becomes production.
Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~021882464639041983694
  • Upwork Job ID: 1882464639041983694
  • Last Price Increase: 2025-01-23
Issue OwnerCurrent Issue Owner: @Christinadobrzyn

Activity

  1. melvin-bot commented on Jan 17, 2025

    @melvin-bot

    Triggered auto assignment to @Christinadobrzyn (NewFeature), see https://stackoverflowteams.com/c/expensify/questions/14418#:~:text=BugZero%20process%20steps%20for%20feature%20requests for more details. Please add this Feature request to a GH project, as outlined in the SO.

  2. melvin-bot commented on Jan 17, 2025

    @melvin-bot

    ⚠️ It looks like this issue is labelled as a New Feature but not tied to any GitHub Project. Keep in mind that all new features should be tied to GitHub Projects in order to properly track external CAP software time ⚠️

  3. mountiny commented on Jan 17, 2025

    @mountiny
    ContributorAuthor

    @Expensify/mobile-deployers Curious for your feedback

  4. melvin-bot commented on Jan 17, 2025

    @melvin-bot

    Triggered auto assignment to Design team member for new feature review - @shawnborton (NewFeature)

  5. added
    InternalRequires API changes or must be handled by Expensify staff
    on Jan 17, 2025
  6. luacmartins commented on Jan 17, 2025

    @luacmartins
    Contributor

    That makes sense to me

  7. 90 remaining items

  8. melvin-bot commented on Mar 27, 2025

    @melvin-bot

    The solution for this issue has been 🚀 deployed to production 🚀 in version 9.1.19-5 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

    If no regressions arise, payment will be issued on 2025-04-03. 🎊

    For reference, here are some details about the assignees on this issue:

    • @eh2077 requires payment through NewDot Manual Requests
  9. melvin-bot commented on Mar 27, 2025

    @melvin-bot

    BugZero Checklist: The PR adding this new feature has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

    • [@eh2077] Please propose regression test steps to ensure the new feature will work correctly on production in further releases.
    • [@Christinadobrzyn] Link the GH issue for creating/updating the regression test once above steps have been agreed upon.
  10. Christinadobrzyn commented on Mar 27, 2025

    @Christinadobrzyn
    Contributor

    Just a heads up that I'm going to be ooo March 27th - April 6th. Back on the 7th.

    I'm going to assign a teammate to pay this out.

    We're waiting on the regression test and payment summary here. #55403 (comment)

  11. removed their assignment
    on Mar 27, 2025
  12. added
    BugSomething is broken. Auto assigns a BugZero manager.
    on Mar 27, 2025
  13. melvin-bot commented on Mar 27, 2025

    @melvin-bot

    Triggered auto assignment to @joekaufmanexpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

  14. Christinadobrzyn commented on Mar 27, 2025

    @Christinadobrzyn
    Contributor

    Wait sorry I just noticed this is closed, @eh2077 are we not paying you out in this? cc @roryabraham does this need payment?

  15. eh2077 commented on Mar 27, 2025

    @eh2077
    Contributor

    @Christinadobrzyn No payment needed here as it doesn't need c+ work, thanks!

  16. Christinadobrzyn commented on Mar 27, 2025

    @Christinadobrzyn
    Contributor

    Ah okay! Thank you! closing again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

Awaiting PaymentAuto-added when associated PR is deployed to productionBugSomething is broken. Auto assigns a BugZero manager.DailyKSv2Fire-CleanupSomething needs to be done after a fireNewFeatureSomething to build that is a new item.

Type

No type

Projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions