-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[$500] [LOW] Tags: Recently selected tag not displayed in recents list after editing an IOU #35506
Comments
Job added to Upwork: https://www.upwork.com/jobs/~014ee0fe0e10710c59 |
Triggered auto assignment to @peterdbarkerUK ( |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @mollfpr ( |
We think that this bug might be related to #wave6 |
ProposalPlease re-state the problem that we are trying to solve in this issue.Latest Selected Tag is not displayed in recents list when we edit it What is the root cause of that problem?We do not update the Tag value to recent as we do with category What changes do you think we should make in order to solve the problem?Update the Tag value to recents What alternative solutions did you explore? (Optional)N/A |
Hmm, is this the behavior for categories that a category selected via edit is added to Recents? I'm not sure if that actually is the expected behavior |
I suppose that makes sense though, it was actually used for the expense in the end. I'll add it to the project board, low prio though |
how to make a collect workspace? and also how to connect to OD? Any articles or place where I can refer and do it? |
Current assignee @mollfpr is eligible for the Internal assigner, not assigning anyone new. |
I think we should hold this issue until #34612 is done |
Triggered auto assignment to @trjExpensify ( |
It probably makes sense @DylanDylann, but mind including your reasoning when making a suggestion to put something on hold? Thanks! |
In #34612, we will migrate to use IOURequestStepTags for all Tag pages so I think the logic update recant tag will be updated for all flows |
Thanks, helpful. Popping it on hold :) |
@DylanDylann, looks like the PR associated with #34612 was deployed to prod, right? Is that it or is there more to do? |
Will check on Feb 15 |
Still can reproduce Screen.Recording.2024-02-15.at.12.16.05.movThe RCA is that After turning off the Internet, everything works well in all flows (create/edit request money, split bill). So I think it is a BE bug. cc @mollfpr |
Okay cool, I'll take this off-hold. @amyevans over to you then if this is BE related, I guess! |
@amyevans @trjExpensify @mollfpr 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! |
PRs up: https://github.com/Expensify/Auth/pull/10019, https://github.com/Expensify/Web-Expensify/pull/41005. Also just a heads up I'll be OOO next week! |
Everything has deployed, closing! |
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: 1.4.34.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:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:
Precondition:
User is employee of Collect workspace
Steps:
Expected Result:
Tag 2 should be present in recent tags list
Actual Result:
Tag 2 is not present in recent tags lis
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6362752_1706727433731.Recording__1984.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: