-
Notifications
You must be signed in to change notification settings - Fork 3.2k
[Due for payment 2025-04-25] Disable resolved ConciergeCategoryOptions actions instead of hiding them #60136
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
Comments
Triggered auto assignment to @Christinadobrzyn ( |
|
Triggered auto assignment to Design team member for new feature review - @dannymcclain ( |
Ah woops, I realize that this was actually part of the original design. @dannymcclain please ignore this! |
Job added to Upwork: https://www.upwork.com/jobs/~021910750263507700710 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @ikevin127 ( |
I'll tackle this myself |
Haha yup! No worries though! |
Job added to Upwork: https://www.upwork.com/jobs/~021911869023812459029 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @jjcoffee ( |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.1.29-10 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-25. 🎊 |
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:
|
@jasperhuangg Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
Regression tests already exist for this! |
No payment needed as no C+ was used for this issue, closing |
Uh oh!
There was an error while loading. Please reload this page.
Problem
The UX of hiding resolved ConciergeCategoryOptions actions feels weird right now. They just suddenly disappear and the responses to them have no context.
Solution
Rather than hiding them, just disable them by either greying them out or turning the buttons into text.
NOTE: To the contributor/C+ working on this, please be requested to be added to the beta.
You can check if the ConciergeCategoryOptions action is resolved by using
ReportActionUtils.isResolvedConciergeCategoryOptions
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @jjcoffeeThe text was updated successfully, but these errors were encountered: