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

[Search v2.1] - There is a Join button in the paid invoice report when accessed from Search #49284

Closed
6 tasks done
izarutskaya opened this issue Sep 16, 2024 · 8 comments
Closed
6 tasks done
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Reviewing Has a PR in review

Comments

@izarutskaya
Copy link

izarutskaya commented Sep 16, 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!


Version Number: 9.0.35-4
Reproducible in staging?: y
Reproducible in production?: y
Email or phone of affected tester (no customers): applausetester+kh010901@applause.expensifail.com
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team

Action Performed:

Precondition:

  • User has received invoice.
  1. Go to staging.new.expensify.com
  2. Log in as invoice receiver.
  3. Go to invoice room.
  4. Pay the invoice
  5. Go to Search > Invoices.
  6. Click on the paid invoice.

Expected Result:

There will not be a Join button in the report.

Actual Result:

There is a Join button in the paid invoice report when accessed from Search.

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

Bug6605311_1726482324067.20240916_182304.mp4

View all open jobs on GitHub

Issue OwnerCurrent Issue Owner: @luacmartins
@izarutskaya izarutskaya added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Sep 16, 2024
Copy link

melvin-bot bot commented Sep 16, 2024

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

@MuaazArshad
Copy link
Contributor

Proposal

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

There is a Join button in the paid invoice report when accessed from Search

What is the root cause of that problem?

We are not checking if the invoice is open or closed.

{!isLoading && canJoin && shouldUseNarrowLayout && <View style={[styles.ph5, styles.pb2]}>{joinButton}</View>}

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

Adding a check for open or closed invoices will resolve the issue.

{!isLoading && canJoin && shouldUseNarrowLayout && <View style={[styles.ph5, styles.pb2]}>{joinButton}</View>}

const isOpenInvoiceReport = ReportUtils.isOpenInvoiceReport(report);

and

{!isLoading && canJoin && shouldUseNarrowLayout && isOpenInvoiceReport && <View style={[styles.ph5, styles.pb2]}>{joinButton}</View>}

@melvin-bot melvin-bot bot added the Overdue label Sep 18, 2024
@luacmartins
Copy link
Contributor

I think this might be solved by this PR #49241

@luacmartins luacmartins changed the title Search - There is a Join button in the paid invoice report when accessed from Search [HOLD # 49241] Search - There is a Join button in the paid invoice report when accessed from Search Sep 19, 2024
@luacmartins luacmartins changed the title [HOLD # 49241] Search - There is a Join button in the paid invoice report when accessed from Search [HOLD # 49241] [Search v2.1] - There is a Join button in the paid invoice report when accessed from Search Sep 19, 2024
Copy link

melvin-bot bot commented Sep 19, 2024

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

@johncschuster
Copy link
Contributor

This is on hold for #49241

@melvin-bot melvin-bot bot removed the Overdue label Sep 19, 2024
@johncschuster johncschuster added the Reviewing Has a PR in review label Sep 19, 2024
@luacmartins
Copy link
Contributor

Hmm I tried testing this again, but for some reason the invoices are not showing up on the Search page at all. @johncschuster could you try the reproducible steps in the OP as well?

@luacmartins
Copy link
Contributor

Actually, I spot the issue in dev and put up a PR to fix it https://github.com/Expensify/Auth/pull/12561

@luacmartins luacmartins self-assigned this Sep 20, 2024
@luacmartins luacmartins changed the title [HOLD # 49241] [Search v2.1] - There is a Join button in the paid invoice report when accessed from Search [Search v2.1] - There is a Join button in the paid invoice report when accessed from Search Sep 20, 2024
@luacmartins
Copy link
Contributor

This is fixed.

@github-project-automation github-project-automation bot moved this from Polish to Done in [#whatsnext] #expense Sep 23, 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. Daily KSv2 Reviewing Has a PR in review
Projects
Status: Done
Development

No branches or pull requests

4 participants