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

[$250] Tag - Dependent tag violation message isn't changing to "all tags required" #43574

Closed
1 of 6 tasks
kavimuru opened this issue Jun 12, 2024 · 43 comments
Closed
1 of 6 tasks
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. External Added to denote the issue can be worked on by a contributor Weekly KSv2

Comments

@kavimuru
Copy link

kavimuru commented Jun 12, 2024

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


Issue found when validating #40741
Version Number: 1.4.82-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: Import the attached dependent_tags.csv in OD for a workspace. Enable multi level tags.

  1. Navigate to https://staging.new.expensify.com/

  2. Log in with an expensifail account

  3. Navigate to the workspace chat

  4. Start submitting a manual expense

  5. Only input the "Merchant" field on the confirmation page

  6. Submit the expense

  7. Open the expense

  8. Click on any of the three missing tags

  9. Choose any of the options

Expected Result:

The message should change to "all tags required" for the reported missing tags.

Actual Result:

Dependent tag violation message isn't changing to "all tags required" for the rest of the tags when there are multiple missing tags and one is selected.

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

Bug6510487_1718181753132!dependent_tags.csv

Bug6510487_1718181753145.bandicam_2024-06-12_10-20-07-860.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~013b410860cea8f950
  • Upwork Job ID: 1801658425666734007
  • Last Price Increase: 2024-08-02
Issue OwnerCurrent Issue Owner: @yuwenmemon
@kavimuru kavimuru added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Jun 12, 2024
Copy link

melvin-bot bot commented Jun 12, 2024

Triggered auto assignment to @twisterdotcom (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

@kavimuru
Copy link
Author

@twisterdotcom 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.

@kavimuru
Copy link
Author

We think this bug might be related to #wave-collect - Release 1

@twisterdotcom
Copy link
Contributor

Also got this: #38682.

@twisterdotcom twisterdotcom added the External Added to denote the issue can be worked on by a contributor label Jun 14, 2024
@melvin-bot melvin-bot bot changed the title Tag - Dependent tag violation message isn't changing to "all tags required" [$250] Tag - Dependent tag violation message isn't changing to "all tags required" Jun 14, 2024
Copy link

melvin-bot bot commented Jun 14, 2024

Job added to Upwork: https://www.upwork.com/jobs/~013b410860cea8f950

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

melvin-bot bot commented Jun 14, 2024

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

Copy link

melvin-bot bot commented Jun 17, 2024

@twisterdotcom, @sobitneupane 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 Jun 17, 2024
@sobitneupane
Copy link
Contributor

Waiting on proposal

@melvin-bot melvin-bot bot removed the Overdue label Jun 18, 2024
Copy link

melvin-bot bot commented Jun 21, 2024

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

@melvin-bot melvin-bot bot added the Overdue label Jun 21, 2024
Copy link

melvin-bot bot commented Jun 21, 2024

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

@sobitneupane
Copy link
Contributor

Waiting for proposal

@melvin-bot melvin-bot bot removed the Overdue label Jun 24, 2024
Copy link

melvin-bot bot commented Jun 26, 2024

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

@melvin-bot melvin-bot bot added the Overdue label Jun 26, 2024
@twisterdotcom
Copy link
Contributor

Still waiting on proposals. May be addressed eventually with Control functionality.

@sobitneupane
Copy link
Contributor

Waiting for proposal

@melvin-bot melvin-bot bot removed the Overdue label Jun 27, 2024
Copy link

melvin-bot bot commented Jun 28, 2024

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

Copy link

melvin-bot bot commented Jul 2, 2024

@twisterdotcom, @sobitneupane Eep! 4 days overdue now. Issues have feelings too...

Copy link

melvin-bot bot commented Jul 26, 2024

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

@melvin-bot melvin-bot bot added the Overdue label Jul 30, 2024
@twisterdotcom
Copy link
Contributor

Still waiting for proposals

@melvin-bot melvin-bot bot removed the Overdue label Jul 31, 2024
@twisterdotcom twisterdotcom added the retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause label Jul 31, 2024
Copy link

melvin-bot bot commented Aug 2, 2024

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

@mvtglobally
Copy link

Issue is reproducible during KI retests.

1722797814057.missing_tags.mp4

@melvin-bot melvin-bot bot added the Overdue label Aug 9, 2024
@mvtglobally mvtglobally removed the retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause label Aug 9, 2024
@teneeto
Copy link
Contributor

teneeto commented Aug 9, 2024

Hey! I'm Eto Olei from Callstack. I will like to look into this issue. Please assign it to me. Thanks!

@melvin-bot melvin-bot bot removed Help Wanted Apply this label when an issue is open to proposals by contributors Overdue labels Aug 9, 2024
@teneeto
Copy link
Contributor

teneeto commented Aug 12, 2024

Hi All!,
One little concern here; in trying to reproduce this issue, I'm wondering what expensify account gives the extra tags, as state, region and city after submitting a manual expense.

After submitting an expense with my account, I'm not getting those fields. what am I missing?

@twisterdotcom
Copy link
Contributor

Hold that thought for now @teneeto. I think we may have somebody internally working on Dependent tags soon.

@teneeto
Copy link
Contributor

teneeto commented Aug 13, 2024

Alright @twisterdotcom, I'll be glad if you Keep me posted on this. Also, I'm still keeping this on the watch, let me know in any way I can assist. Thanks

@yuwenmemon yuwenmemon self-assigned this Aug 17, 2024
@melvin-bot melvin-bot bot added the Overdue label Aug 26, 2024
@yuwenmemon
Copy link
Contributor

Will look at this as a part of the tags project soon.

@melvin-bot melvin-bot bot removed the Overdue label Aug 28, 2024
@teneeto
Copy link
Contributor

teneeto commented Aug 29, 2024

Cool, @yuwenmemon, I'm still here; please let me know if I can come in for any help.

@melvin-bot melvin-bot bot added the Overdue label Sep 6, 2024
@twisterdotcom
Copy link
Contributor

This is going through the motions internally now.

@melvin-bot melvin-bot bot removed the Overdue label Sep 9, 2024
@melvin-bot melvin-bot bot added the Overdue label Sep 17, 2024
@yuwenmemon
Copy link
Contributor

I don't get it.... this happens for independent tags too. Are we sure we need this to say "all tags required"? cc @cead22

Kapture.2024-09-17.at.23.16.26.mp4

@melvin-bot melvin-bot bot removed the Overdue label Sep 18, 2024
@twisterdotcom
Copy link
Contributor

Hmm good point, I think given we say "Missing tagName" for each level, this is fine now.

@yuwenmemon
Copy link
Contributor

Sweet - closing!

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. External Added to denote the issue can be worked on by a contributor Weekly KSv2
Projects
Status: Done
Development

No branches or pull requests

6 participants