Skip to content

Deploy Checklist: Expensify.cash 2021-07-28 #4279

Closed

Description

@OSBotify

Release Version: 1.0.81-4
Compare Changes: production...staging

This release contains changes from the following pull requests:

Deploy Blockers:

cc @Expensify/applauseleads

Activity

  1. isagoico commented on Jul 28, 2021

    @isagoico

    Starting QA 🚀

  2. isagoico commented on Jul 29, 2021

    @isagoico

    #4293 just confirmed there was no crash 🎉 checking it off.

    There was a bunch of blocked steps on our regression because of this issue 🔝. I would propose restarting QA by removing/readding the Lock label and proceed with a new QA run so we have a full regression and test new PRs OR we could just retest the blocked steps (that are a lot) only. What do you think @roryabraham @AndrewGable ?

    Side note: Looks like the CP staging bug is back, these PRs are not in staging and need to be checked off/removed:

    1. Adjust outline for focused elments #4170
    2. Context-menu repositioned #4194
    3. Added Balance badge to the Settings page #4220
    4. Revert add payment button changes #4242
    5. Fix bolded text for Existing Owners Error #4261
    6. Prevent large attachment #4269
    7. Use proper width for Cancel/Decline Button Text #4275
    8. [No QA] Setup Flipper plugins when on native & dev #4290
    9. [No QA] Fix crash #4297
  3. roryabraham commented on Jul 29, 2021

    @roryabraham
    Contributor

    Let's retest the blocked steps and do another full regression suite tomorrow. But there should be no need to remove and re-add the lock label, right? We don't need to potentially introduce more deploy blockers in this release 😅

  4. isagoico commented on Jul 30, 2021

    @isagoico

    These 2 PRs had an issue are still failing:
    #4190 (comment) (@johnmlee101)
    #4240 (comment) (@Dal-Papa)

    This internal QA need to be tested too: #4227 (@HorusGoul @Dal-Papa)

  5. isagoico commented on Jul 30, 2021

    @isagoico

    #4263 was retested and was a pass 🎉

  6. Jag96 commented on Jul 30, 2021

    @Jag96
    Contributor

    Tested and checked off #4227, and just CP'd #4322 to staging so that should be good to re-test once it gets deployed. Will look into #4240 (comment) next.

  7. Jag96 commented on Jul 30, 2021

    @Jag96
    Contributor

    Checked off the following that were added by the CP bug:

    • 4090
    • 4172
    • 4272
    • 4285
    • 4294
    • 4295
    • 4301
    • 4309
    • 4311
    • 4312
    • 4313
    • 4317
    • 4318
    • 4322 (This was the fix for 4190)
  8. Jag96 commented on Jul 30, 2021

    @Jag96
    Contributor

    Looking at #4240 I can't reproduce the issue on staging on web:

    Account 1 Account 2
    image image

    @isagoico are you still seeing the issue on staging?

    Edit: Ah nvm, looks like its only on mobile/desktop, will re-test

  9. isagoico commented on Jul 30, 2021

    @isagoico

    #4190 (comment) checking it off 🎉

  10. Jag96 commented on Jul 30, 2021

    @Jag96
    Contributor

    Ok, tested on staging iOS and Desktop and it's working for me!

    iOS Desktop
    Image from iOS (3) image

    For desktop, I had to go to staging.new.expensify.com/Expensify.cash.dmg to ensure I was on staging. @isagoico can you reproduce #4240 on staging still?

  11. isagoico commented on Jul 30, 2021

    @isagoico

    @Jag96 asked a tester to check on their side, will update with the results

  12. isagoico commented on Jul 30, 2021

    @isagoico

    image
    It was a pass for the tester 🎉 checking it off.
    This should be the last to check off the list, will go ahead with the prod deploy CC @roryabraham

  13. isagoico commented on Jul 30, 2021

    @isagoico

    :shipit:

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

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions