-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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 is listing PRs from 2-3 previous builds #24030
Comments
Triggered auto assignment to @Christinadobrzyn ( |
Bug0 Triage Checklist (Main S/O)
|
Not sure if this should be external - I assume it needs to be internal? Going to add internal and we can adjust if needed. |
Job added to Upwork: https://www.upwork.com/jobs/~0158becf89864f9a24 |
Triggered auto assignment to Contributor Plus for review of internal employee PR - @0xmiroslav ( |
@0xmiroslav can you take a peek if this should be internal or external when you have time? Thanks! |
@Christinadobrzyn this should be internal. Can you please add |
Triggered auto assignment to @puneetlath ( |
Thanks for confirming @0xmiroslav added the engineering label |
@puneetlath, @Christinadobrzyn, @0xmiroslav Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
looks like #23179 might be related. @0xmiroslav feel free to add any updates on this one! |
#23179 doesn't help. It's just one example affected by this issue. |
Not sure about an update on this one - @0xmiroslav what questions do you have for @puneetlath? Can you provide them here or in a Slack convo? |
Just nudging this for an update @puneetlath or @0xmiroslav? |
@puneetlath, @Christinadobrzyn, @0xmiroslav Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
1 similar comment
@puneetlath, @Christinadobrzyn, @0xmiroslav Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
@mvtglobally can you please tag relevant people to push this forward? I don't have access to slack links in OP. |
Reached out in these chats to try and figure out what we are doing with this: https://expensify.slack.com/archives/C9YU7BX5M/p1692730312167289?thread_ts=1690509355.423659&cid=C9YU7BX5M |
@puneetlath @Christinadobrzyn @0xmiroslav this issue is now 3 weeks old. There is one more week left before this issue breaks WAQ. What needs to happen to get a PR in review this week? Please create a thread in #expensify-open-source to discuss. Thanks! |
Sorry, I've not been great about trying to figure out how to guide this issue. I reached out to the team in Slack to see if I can get some guidance. |
Just going to make sure this is still a thing as I haven't heard more about it recently - @mvtglobally are you still seeing this issue? Slack - https://expensify.slack.com/archives/C9YU7BX5M/p1693345044647759 |
still happening on the latest checklist #26113 (comment) |
cc @roryabraham |
Closing this as a dupe of #21763 |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Action Performed:
Expected Result:
Only PRs deployed in the build should be listed
CPs should be added to the checklist once deployed
Actual Result:
Multiple PRs from previous deploys displayed in the checklist even after they were already deployed to PROD.
Once CP is deployed, its not added to the checklist
Workaround:
unknown
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
Issue reported by: Applause Testing Team
Slack conversation:
https://expensify.slack.com/archives/C9YU7BX5M/p1690509355423659
https://expensify.slack.com/archives/C9YU7BX5M/p1690256275653639
https://expensify.slack.com/archives/C9YU7BX5M/p1689965139270719
https://expensify.slack.com/archives/C9YU7BX5M/p1690935729223149
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: