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 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.

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

6 participants