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] [Wave 6: Tags] Expense - Selected subcategory shows different view in Tag & Category #32489

Closed
6 tasks done
lanitochka17 opened this issue Dec 5, 2023 · 21 comments
Closed
6 tasks done
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 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

Comments

@lanitochka17
Copy link

lanitochka17 commented Dec 5, 2023

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!


Version Number: 1.4.8-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: The workspace chat has some tags in Parent: Child format

  1. Go to workspace chat
  2. Click + > Request money > Manual
  3. Enter amount > Continue
  4. Click Show more > Tag
  5. Select a subcategory
  6. Click Tag
  7. Repeat Step 1 to 6 with Category

Expected Result:

The selected subcategory will show the same view in Tag and Category

Actual Result:

The selected subcategory does not show the same view in Tag and Category
For Tag, the selected subcategory is in hierarchy order
For Category, the selected subcategory is in Parent: Child format

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

Bug6301777_1701779561826.bandicam_2023-12-05_15-18-59-178.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~019f4d74182e84937e
  • Upwork Job ID: 1732061052657393664
  • Last Price Increase: 2023-12-12
@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 Dec 5, 2023
@melvin-bot melvin-bot bot changed the title Expense - Selected subcategory shows different view in Tag & Category [$500] Expense - Selected subcategory shows different view in Tag & Category Dec 5, 2023
Copy link

melvin-bot bot commented Dec 5, 2023

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

Copy link

melvin-bot bot commented Dec 5, 2023

Job added to Upwork: https://www.upwork.com/jobs/~019f4d74182e84937e

@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Dec 5, 2023
Copy link

melvin-bot bot commented Dec 5, 2023

Bug0 Triage Checklist (Main S/O)

  • This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
  • This bug is not a duplicate report (check E/App issues and #expensify-bugs)
    • If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead
  • This bug is reproducible using the reproduction steps in the OP. S/O
    • If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
    • If the reproduction steps aren't clear and you determine the correct steps, please update the OP.
  • This issue is filled out as thoroughly and clearly as possible
    • Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.
  • I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

Copy link

melvin-bot bot commented Dec 5, 2023

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

@happy-devs
Copy link

Proposal

Please re-state the problem that we are trying to solve in this issue.

When requesting money manually, the list options for Tags and Categories do use a uniform format for displaying selected subcategories.

What is the root cause of that problem?

The file OptionsListUtils has to be changed to fix the rendering of child and parent for tags.

What changes do you think we should make in order to solve the problem?

Update the OptionsListUtils functions to ensure that both Tags and Categories use the same format.

What alternative solutions did you explore? (Optional)

Reminder: Please use plain English, be brief and avoid jargon. Feel free to use images, charts or pseudo-code if necessary. Do not post large multi-line diffs or write walls of text. Do not create PRs unless you have been hired for this job.

@c3024
Copy link
Contributor

c3024 commented Dec 5, 2023

Proposal

Please re-state the problem that we are trying to solve in this issue.

Selected subcateogry shows differently in Category and Tag.

What is the root cause of that problem?

We are passing true here for isOneLine

data: getIndentedOptionTree(selectedOptions, true),

so it is shown in one line whereas for tags we use no flag so the default flag is false

What changes do you think we should make in order to solve the problem?

We should remove the flag for isOneLine here if we want to show category options also in hierarchical view

data: getIndentedOptionTree(selectedOptions, true),

If we want to show hierarchical view everywhere we can remove all true flags like here for search
data: getIndentedOptionTree(searchCategories, true),

On the other hand if we like to follow the view of category for tags, we pass the true flag for tags here

data: getTagsOptions(selectedTagOptions),

and here
data: getTagsOptions(searchTags),

What alternative solutions did you explore? (Optional)

@tienifr
Copy link
Contributor

tienifr commented Dec 5, 2023

Proposal

Please re-state the problem that we are trying to solve in this issue.

The selected subcategory does not show the same view in Tag and Category
For Tag, the selected subcategory is in hierarchy order
For Category, the selected subcategory is in Parent: Child format

What is the root cause of that problem?

We're not using isOneLine true in tags

What changes do you think we should make in order to solve the problem?

Use isOneLine true in tags

What alternative solutions did you explore? (Optional)

NA

@shahinyan11
Copy link
Contributor

@lanitochka17 @c3024 @tienifr @happy-devs I don't see anything after the category. Could you tell me what I did wrong?
Screenshot 2023-12-05 at 20 04 53

@happy-devs
Copy link

@shahinyan11 looks like this is only in the dev environment, not the https://new.expensify.com portal

@c3024
Copy link
Contributor

c3024 commented Dec 5, 2023

Please follow this.

@FitseTLT
Copy link
Contributor

FitseTLT commented Dec 6, 2023

already fixed

@melvin-bot melvin-bot bot added the Overdue label Dec 8, 2023
@CortneyOfstad
Copy link
Contributor

@thesahindia can you confirm if this was fixed via the linked PR above? TIA!

@melvin-bot melvin-bot bot removed the Overdue label Dec 8, 2023
@amyevans amyevans changed the title [$500] Expense - Selected subcategory shows different view in Tag & Category [Wave 6: Tags][$500] Expense - Selected subcategory shows different view in Tag & Category Dec 8, 2023
@melvin-bot melvin-bot bot added the Overdue label Dec 11, 2023
@CortneyOfstad
Copy link
Contributor

Bump @thesahindia ^^^ Thanks!

@melvin-bot melvin-bot bot removed the Overdue label Dec 11, 2023
Copy link

melvin-bot bot commented Dec 12, 2023

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

@kbecciv
Copy link

kbecciv commented Dec 14, 2023

@CortneyOfstad Issue is still reproducible

20231214_215846.mp4

@CortneyOfstad
Copy link
Contributor

Thanks @kbecciv! Will keep this open while we wait for proposals!

@melvin-bot melvin-bot bot removed the Overdue label Dec 14, 2023
Copy link

melvin-bot bot commented Dec 18, 2023

@CortneyOfstad, @thesahindia Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

@melvin-bot melvin-bot bot added the Overdue label Dec 18, 2023
Copy link

melvin-bot bot commented Dec 18, 2023

@CortneyOfstad, @thesahindia Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

@greg-schroeder greg-schroeder changed the title [Wave 6: Tags][$500] Expense - Selected subcategory shows different view in Tag & Category [$500] [Wave 6: Tags] Expense - Selected subcategory shows different view in Tag & Category Dec 19, 2023
Copy link

melvin-bot bot commented Dec 19, 2023

@CortneyOfstad @thesahindia this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

@rezkiy37
Copy link
Contributor

Hi, I’m Michael (Mykhailo) from Callstack and I would like to work on this issue.

@CortneyOfstad
Copy link
Contributor

confirmed here that the features are actually working as designed. The tags do not interact the same as the categories in this instance, so the issue can be closed 👍

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. Daily KSv2 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
Projects
No open projects
Development

No branches or pull requests

10 participants