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

[$500] [LOW] Tags: Recently selected tag not displayed in recents list after editing an IOU #35506

Closed
3 of 6 tasks
lanitochka17 opened this issue Jan 31, 2024 · 21 comments
Closed
3 of 6 tasks
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Engineering Internal Requires API changes or must be handled by Expensify staff Reviewing Has a PR in review Weekly KSv2

Comments

@lanitochka17
Copy link

lanitochka17 commented Jan 31, 2024

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:

  1. Employee navigate to Workspace chat
  2. Go to +> Request money> Manual> Select tag 1> Complete the flow
  3. Navigate to created IOU> Edit tag> Select tag 2> Save
  4. Navigate to workspace chat
  5. Go to +> Request money> Manual> Tag

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?

  • Android: Native
  • Android: mWeb Chrome
  • iOS: Native
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

Bug6362752_1706727433731.Recording__1984.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~014ee0fe0e10710c59
  • Upwork Job ID: 1752771933542309888
  • Last Price Increase: 2024-01-31
@lanitochka17 lanitochka17 added External Added to denote the issue can be worked on by a contributor Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Jan 31, 2024
@melvin-bot melvin-bot bot changed the title IOU - Tag is not displayed in recents list when change it on created IOU [$500] IOU - Tag is not displayed in recents list when change it on created IOU Jan 31, 2024
Copy link

melvin-bot bot commented Jan 31, 2024

Job added to Upwork: https://www.upwork.com/jobs/~014ee0fe0e10710c59

Copy link

melvin-bot bot commented Jan 31, 2024

Triggered auto assignment to @peterdbarkerUK (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Jan 31, 2024
Copy link

melvin-bot bot commented Jan 31, 2024

Triggered auto assignment to Contributor-plus team member for initial proposal review - @mollfpr (External)

@lanitochka17
Copy link
Author

We think that this bug might be related to #wave6
CC @greg-schroeder

@allgandalf
Copy link
Contributor

allgandalf commented Jan 31, 2024

Proposal

Please 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

@greg-schroeder
Copy link
Contributor

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

@greg-schroeder
Copy link
Contributor

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

@greg-schroeder greg-schroeder changed the title [$500] IOU - Tag is not displayed in recents list when change it on created IOU [$500] [LOW] Tags: Recently selected tag not displayed in recents list after editing an IOU Jan 31, 2024
@vishrutagarwal97
Copy link

how to make a collect workspace? and also how to connect to OD? Any articles or place where I can refer and do it?

@amyevans amyevans self-assigned this Jan 31, 2024
@greg-schroeder greg-schroeder added Internal Requires API changes or must be handled by Expensify staff and removed External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors labels Feb 1, 2024
Copy link

melvin-bot bot commented Feb 1, 2024

Current assignee @mollfpr is eligible for the Internal assigner, not assigning anyone new.

@DylanDylann
Copy link
Contributor

I think we should hold this issue until #34612 is done

@kadiealexander kadiealexander added Bug Something is broken. Auto assigns a BugZero manager. and removed Bug Something is broken. Auto assigns a BugZero manager. labels Feb 1, 2024
Copy link

melvin-bot bot commented Feb 1, 2024

Triggered auto assignment to @trjExpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

@trjExpensify
Copy link
Contributor

I think we should hold this issue until #34612 is done

It probably makes sense @DylanDylann, but mind including your reasoning when making a suggestion to put something on hold? Thanks!

@DylanDylann
Copy link
Contributor

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

@trjExpensify trjExpensify changed the title [$500] [LOW] Tags: Recently selected tag not displayed in recents list after editing an IOU [Hold #34612] [$500] [LOW] Tags: Recently selected tag not displayed in recents list after editing an IOU Feb 2, 2024
@trjExpensify trjExpensify added Weekly KSv2 and removed Daily KSv2 labels Feb 2, 2024
@trjExpensify
Copy link
Contributor

Thanks, helpful. Popping it on hold :)

@melvin-bot melvin-bot bot added the Overdue label Feb 12, 2024
@trjExpensify
Copy link
Contributor

@DylanDylann, looks like the PR associated with #34612 was deployed to prod, right? Is that it or is there more to do?

@melvin-bot melvin-bot bot removed the Overdue label Feb 12, 2024
@DylanDylann
Copy link
Contributor

Will check on Feb 15

@DylanDylann
Copy link
Contributor

Still can reproduce

Screen.Recording.2024-02-15.at.12.16.05.mov

The RCA is that OpenDraftWorkspaceRequest API return wrong data

Screenshot 2024-02-15 at 12 18 39

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

@melvin-bot melvin-bot bot added the Overdue label Feb 20, 2024
@trjExpensify trjExpensify changed the title [Hold #34612] [$500] [LOW] Tags: Recently selected tag not displayed in recents list after editing an IOU [$500] [LOW] Tags: Recently selected tag not displayed in recents list after editing an IOU Feb 21, 2024
@trjExpensify
Copy link
Contributor

Okay cool, I'll take this off-hold. @amyevans over to you then if this is BE related, I guess!

@melvin-bot melvin-bot bot removed the Overdue label Feb 21, 2024
Copy link

melvin-bot bot commented Feb 21, 2024

@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!

@amyevans
Copy link
Contributor

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!

@melvin-bot melvin-bot bot removed the Overdue label Feb 23, 2024
@amyevans amyevans added Reviewing Has a PR in review Weekly KSv2 and removed Daily KSv2 labels Feb 23, 2024
@amyevans
Copy link
Contributor

amyevans commented Mar 6, 2024

Everything has deployed, closing!

@amyevans amyevans closed this as completed Mar 6, 2024
@github-project-automation github-project-automation bot moved this from Polish to Done in [#whatsnext] #wave-collect Mar 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Engineering Internal Requires API changes or must be handled by Expensify staff Reviewing Has a PR in review Weekly KSv2
Projects
No open projects
Development

No branches or pull requests

10 participants