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

Invoices Category visible for invoice recipient in the Search LHN #52379

Closed
3 of 8 tasks
IuliiaHerets opened this issue Nov 12, 2024 · 4 comments
Closed
3 of 8 tasks

Invoices Category visible for invoice recipient in the Search LHN #52379

IuliiaHerets opened this issue Nov 12, 2024 · 4 comments
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2

Comments

@IuliiaHerets
Copy link

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.60-0
Reproducible in staging?: Y
Reproducible in production?: Y
Issue was found when executing this PR: #52148
Email or phone of affected tester (no customers): [email protected]
Issue reported by: Applause Internal Team

Action Performed:

Precondition: User signed into account A on the main tab and into account B, which is not a member of any workspace with invoices and has never released files, in the incognito tab

  1. [Account A] open https://staging.new.expensify.com/

  2. [Account A] Create a workspace and enable invoices

  3. [Account B] Navigate to search tab and observe categories

  4. [Account A] Send invoice to account B

  5. [Account B] Repeat step 3

Expected Result:

According the step 10 in the PR, "Veriy that invoices category is not visible in the Search LHN"
(Invoices categories should not be displayed in the search tab for invoice recipient in the Search LHN)

Actual Result:

Invoices Category visible for invoice recipient in the Search LHN

Workaround:

Unknown

Platforms:

  • Android: Standalone
  • Android: HybridApp
  • Android: mWeb Chrome
  • iOS: Standalone
  • iOS: HybridApp
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Bug6661868_1731361613307.Screen_Recording_2024-11-11_at_23.26.59.mp4

View all open jobs on GitHub

@IuliiaHerets IuliiaHerets added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Nov 12, 2024
Copy link

melvin-bot bot commented Nov 12, 2024

Triggered auto assignment to @trjExpensify (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.

@NJ-2020
Copy link
Contributor

NJ-2020 commented Nov 12, 2024

I think we should show the invoice category menu if the user has invoices report
But if we don't want to show the invoice category menu we can simply remove this check hasInvoiceReports()

if (hasWorkspaceWithInvoices(session?.email) || hasInvoiceReports()) {

@rushatgabhane
Copy link
Member

not a bug. it is expected that invoices will be seen if the user has received invoices

@luacmartins
Copy link
Contributor

This is expected. The invoice receiver should be able to filter by invoices since they have invoices in their account. 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. Daily KSv2
Projects
None yet
Development

No branches or pull requests

5 participants