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

Track expense when submitted to the workspace shows "owes" in preview and "tracking" in LHN #56346

Open
8 tasks done
IuliiaHerets opened this issue Feb 4, 2025 · 19 comments
Open
8 tasks done
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Engineering Reviewing Has a PR in review Weekly 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.94-1
Reproducible in staging?: Yes
Reproducible in production?: N/A - new feature, doesn't exist in prod
If this was caught during regression testing, add the test name, ID and link from TestRail: Exp
Email or phone of affected tester (no customers): [email protected]
Issue reported by: Applause Internal Team
Device used: Mac 15.0 / Chrome
App Component: Money Requests

Action Performed:

  1. Go to staging.new.expensify.com
  2. Go to self DM.
  3. Submit a manual expense to self DM.
  4. Click Share it with my accountant.
  5. Select any workspace if there are workspaces, or create a new one.
  6. Enter merchant and submit it to the workspace chat.
  7. Go to workspace chat.
  8. Note that expense preview shows "spent x" and LHN shows "tracking x".
  9. Delete the expense.
  10. Go to self DM and submit a manual expense.
  11. Select Submit it to someone.
  12. Select a workspace and submit it to the workspace chat.
  13. Go to workspace chat.

Expected Result:

If the submitted expense is supposed to be a non-track expense, the expense preview should show "owes x" and LHN should also shows "owes x".

If the submitted expense is supposed to be a track expense, the expense preview should show "spent x" and LHN should also shows "tracking x".

Actual Result:

The expense preview shows "owes x" and LHN shows "tracking x". It is giving unclear indication that the submitted expense could be 50% non-track expense and 50% track expense.

Workaround:

Unknown

Platforms:

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

Screenshots/Videos

Bug6733165_1738675035663.20250204_210942.mp4

View all open jobs on GitHub

@IuliiaHerets IuliiaHerets added Bug Something is broken. Auto assigns a BugZero manager. DeployBlockerCash This issue or pull request should block deployment labels Feb 4, 2025
Copy link

melvin-bot bot commented Feb 4, 2025

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

Copy link

melvin-bot bot commented Feb 4, 2025

Triggered auto assignment to @nkuoch (DeployBlockerCash), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.

@melvin-bot melvin-bot bot added the Daily KSv2 label Feb 4, 2025
Copy link

melvin-bot bot commented Feb 4, 2025

💬 A slack conversation has been started in #expensify-open-source

@github-actions github-actions bot added Engineering Hourly KSv2 and removed Daily KSv2 labels Feb 4, 2025
Copy link
Contributor

github-actions bot commented Feb 4, 2025

👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

  1. Identify the pull request that introduced this issue and revert it.
  2. Find someone who can quickly fix the issue.
  3. Fix the issue yourself.

@Beamanator
Copy link
Contributor

Helping look into this one 🤚

@roryabraham roryabraham assigned Beamanator and unassigned nkuoch Feb 4, 2025
@Beamanator
Copy link
Contributor

Pretty sure this issue comes from #52061, but since this is a new feature, I'm 99% sure we can mark this NAB - confirming in slack before doing that

@luacmartins
Copy link
Contributor

@FitseTLT @hoangzinh It seems like this came from our PR. @FitseTLT could you please look into this? We want to display the ... owes message in the LHN, instead of the tracking one.

@Beamanator Beamanator added Daily KSv2 and removed DeployBlockerCash This issue or pull request should block deployment Hourly KSv2 labels Feb 4, 2025
@FitseTLT
Copy link
Contributor

FitseTLT commented Feb 4, 2025

@FitseTLT @hoangzinh It seems like this came from our PR. @FitseTLT could you please look into this? We want to display the ... owes message in the LHN, instead of the tracking one.

OK on it

@Beamanator
Copy link
Contributor

Marking NAB 👍 @luacmartins you wanna take over here? 🤷 i'm down to keep it, but happy to pass it to you!

@luacmartins
Copy link
Contributor

Taking over

@FitseTLT
Copy link
Contributor

FitseTLT commented Feb 4, 2025

@FitseTLT @hoangzinh It seems like this came from our PR. @FitseTLT could you please look into this? We want to display the ... owes message in the LHN, instead of the tracking one.

@luacmartins The tracking... LHN is being set from BE

2025-02-04.23-07-36.mp4

@luacmartins
Copy link
Contributor

So it seems like the IOU action is still set to type = 'track', so we're fetching it here and we end up translating it here

@luacmartins
Copy link
Contributor

@yuwenmemon @thienlnam do you know if we can update the IOU action to remove the type: 'track' once we call AddTrackedExpenseToPolicy or do we use that for some other tracking purpose?

@thienlnam
Copy link
Contributor

IIRC, AddTrackedExpenseToPolicy is adding a non-reimbursable expense to a workspace so the type track is used to denote that

@luacmartins
Copy link
Contributor

Hmm in this case we're actually passing reimbursable = true to the expense. Should we be changing the type then?

@thienlnam
Copy link
Contributor

Ah given that we're overriding the reimbursable state then we should change the type based on that

@sakluger
Copy link
Contributor

sakluger commented Feb 7, 2025

Just checking, what's the next step here?

@melvin-bot melvin-bot bot added the Overdue label Feb 10, 2025
@sakluger
Copy link
Contributor

@luacmartins friendly bump, what's the next step on this GH issue?

@melvin-bot melvin-bot bot removed the Overdue label Feb 10, 2025
@luacmartins
Copy link
Contributor

I have this PR in review https://github.com/Expensify/Auth/pull/14118 that fixes the issue

@luacmartins luacmartins added Reviewing Has a PR in review Weekly KSv2 and removed Daily KSv2 labels Feb 10, 2025
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. Engineering Reviewing Has a PR in review Weekly KSv2
Projects
None yet
Development

No branches or pull requests

8 participants