Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Deploy Checklist: New Expensify 2021-12-15 #6778

Closed
37 tasks done
OSBotify opened this issue Dec 15, 2021 · 21 comments
Closed
37 tasks done

Deploy Checklist: New Expensify 2021-12-15 #6778

OSBotify opened this issue Dec 15, 2021 · 21 comments
Assignees
Labels
🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging StagingDeployCash

Comments

@OSBotify
Copy link
Contributor

OSBotify commented Dec 15, 2021

Release Version: 1.1.21-1
Compare Changes: production...staging

This release contains changes from the following pull requests:

Internal QA:

Deploy Blockers:

cc @Expensify/applauseleads

@MelvinBot
Copy link

📣 @applausebot You have been assigned to this job by @OSBotify!
Please apply to this job in Upwork and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑‍💻
Keep in mind: Code of Conduct | Contributing 📖

@mvtglobally mvtglobally added the 🔐 LockCashDeploys 🔐 Prevent new code from being deployed to staging label Dec 15, 2021
@mvtglobally
Copy link

Starting QA

@Julesssss
Copy link
Contributor

Julesssss commented Dec 16, 2021

The 'New room - Messages appear at the top of the page in chat room ' blocker should now have been fixed with this PR. It was cherry-picked to staging: #6798

@Julesssss
Copy link
Contributor

I closed the second blocker and checked it off, as the unread indicator feature isn't implemented on Android.

@mvtglobally
Copy link

checking off #6723 based on this #6723 (comment)

@mvtglobally
Copy link

Checking off #6792 its a pass

@mvtglobally
Copy link

mvtglobally commented Dec 16, 2021

@roryabraham @Julesssss I think we are good to close this checklist.

@roryabraham
Copy link
Contributor

roryabraham commented Dec 16, 2021

I am going to hold off and let @Julesssss handle it this time 😉

@Julesssss
Copy link
Contributor

Thanks @mvtglobally & @roryabraham!

@Julesssss
Copy link
Contributor

:shipit:

@OSBotify OSBotify reopened this Dec 17, 2021
@OSBotify
Copy link
Contributor Author

This issue either has unchecked QA steps or has not yet been marked with the :shipit: emoji of approval.
Reopening!

@Julesssss
Copy link
Contributor

Let me try again, I think I had not yet removed the deployBlocker label from all issues.

@Julesssss
Copy link
Contributor

:shipit:

@OSBotify
Copy link
Contributor Author

This issue either has unchecked QA steps or has not yet been marked with the :shipit: emoji of approval.
Reopening!

@OSBotify OSBotify reopened this Dec 17, 2021
@Julesssss
Copy link
Contributor

Okay, so it looks like this PR was added since logging off last night.

@Julesssss
Copy link
Contributor

Julesssss commented Dec 17, 2021

I see, we have a new InternalQA label for NewDot. @nickmurray47 is assigned to review the PR, but if I get through all my other tasks for the day I will take over.

@roryabraham
Copy link
Contributor

@Julesssss you also need to check off all the accessibility checkboxes.

@Julesssss
Copy link
Contributor

Checking off the stagingQA PR as the feature isn't yet live and we will have more testing before releasing the feature.

@Julesssss
Copy link
Contributor

:shipit:

@Julesssss
Copy link
Contributor

I was so excited to release that I forgot it was Friday afternoon 😬

I'll follow the release, hopefully App Review will add some delay.

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

No branches or pull requests

6 participants