-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
[HOLD] [$250] Tags-Deselecting state tag shows state no longer valid error #49504
Comments
Triggered auto assignment to @sakluger ( |
@sakluger FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors |
@IuliiaHerets I think you attached the wrong video, it doesn't match the reproduction steps. Could you please check? |
@sakluger Fixed it, sorry |
Thanks! Adding the external label now. |
Job added to Upwork: https://www.upwork.com/jobs/~021838256634008984824 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @rayane-djouah ( |
The steps for multi-tags setup are here: #44995 (comment) in |
Looking for proposals! |
@sakluger, @rayane-djouah, @IuliiaHerets Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
Waiting on proposals |
Hi, I am Michael (Mykhailo) from Callstack, an expert agency and I can work on this issue. |
📣 @rayane-djouah 🎉 An offer has been automatically sent to your Upwork account for the Reviewer role 🎉 Thanks for contributing to the Expensify app! |
@sakluger, @rezkiy37, @rayane-djouah Whoops! This issue is 2 days overdue. Let's get this updated quick! |
@JmillsExpensify bump on #49504 (comment) |
Yes, I agree with Carlos. Let's put this on hold for @yuwenmemon's dependent tag project. |
@yuwenmemon is there a particular GH issue that I can put this on hold for? Or could we close it because you will naturally build this as part of your dependent tag project? |
Let's put this on HOLD waiting on https://github.com/Expensify/Expensify/issues/418655 And then we can include it as part of the polish issues for that project (there's a chance we just fix it as part of implementation) |
@yuwenmemon I assume this will be purely backend, right? Will anything be needed from me? |
I'm unassigning myself from this issue, assuming it will be handled internally. Please reassign me if any frontend changes are needed. Thanks! |
Sounds good! I'm adding this to the |
The hold issue is also still on hold. |
The hold issue (https://github.com/Expensify/Expensify/issues/418655) is still on hold. |
@sakluger, any updates? More context - #49504 (comment). |
I’m planning a short vacation next week from Monday to Wednesday, from the 17th to the 19th. I am working on the issue later. |
I am back. |
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Version Number: 9.0.38-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4988212
Issue reported by: Applause Internal Team
Action Performed:
Pre-conditions:
Expected Result:
Deselecting state tag must show state tag missing error.
Actual Result:
Deselecting state tag shows state no longer valid error.
Workaround:
Unknown
Platforms:
Screenshots/Videos
Bug6608867_1726749697606.screenrecorder-2024-09-19-13-04-15-98_compress_1.1.mp4
Dependent - Multi Level tags NEW (2).csv
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: