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 v1.2] - No approve button on the transaction in search page #53018

Closed
1 of 8 tasks
lanitochka17 opened this issue Nov 23, 2024 · 6 comments
Closed
1 of 8 tasks
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2

Comments

@lanitochka17
Copy link

lanitochka17 commented Nov 23, 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: 9.0.66-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: N/A
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers): [email protected]
Issue reported by: Applause - Internal Team

Issue found when executing PR #52441

Action Performed:

Pre conditions: Make sure you have a report that can be approved, i.e. create a workspace, invite a member, turn on approvals and reimbursement (workflows), as the member submit a few expenses

  1. As the admin open the expenses submitted and observe the approve button appears
  2. Navigate to search page

Expected Result:

Approve button appears next to the expense/report

Actual Result:

No approve buttons are appearing next to the expense/report

Workaround:

Unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

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

Screenshots/Videos

Add any screenshot/video evidence
Bug6674002_1732356682180.bandicam_2024-11-23_13-05-40-412.mp4

View all open jobs on GitHub

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

melvin-bot bot commented Nov 23, 2024

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

@abzokhattab
Copy link
Contributor

not able to reproduce:
image

@cretadn22
Copy link
Contributor

@luacmartins

This is a BE bug. The BE returns a incorrect value for the isFromOneTransactionReport field

if (!ReportUtils.isMoneyRequestReport(report) || (isTransaction && !data[key].isFromOneTransactionReport)) {
return CONST.SEARCH.ACTION_TYPES.VIEW;
}

As a result, this condition is met, and the View action is displayed.

@luacmartins luacmartins self-assigned this Nov 25, 2024
@luacmartins luacmartins changed the title Search - No approve button on the transaction in search page [Search v1.2] - No approve button on the transaction in search page Nov 25, 2024
@luacmartins
Copy link
Contributor

Thanks! I'll take a look at it

@luacmartins
Copy link
Contributor

Hm this is expected. If the report has multiple expenses, we only show the Approve and Pay button on the "report" views, e.g. Outstanding and not on the All page

@luacmartins
Copy link
Contributor

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
Status: Done
Development

No branches or pull requests

5 participants