Expensify / App

Welcome to New Expensify: a complete re-imagination of financial collaboration, centered around chat. Help us build the next generation of Expensify by sharing feedback and contributing to the code.
https://new.expensify.com
MIT License
3.57k stars 2.91k forks source link

Track expense-In self DM, create expense is shown instead track expense #52897

Open IuliiaHerets opened 17 hours ago

IuliiaHerets commented 17 hours ago

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: v9.0.65-1 Reproducible in staging?: Y Reproducible in production?: Y Isuue was found when executing this PR: https://github.com/Expensify/App/pull/52786 Issue reported by: Applause Internal Team

Action Performed:

  1. Go to https://staging.new.expensify.com/home
  2. Login with new gmail account
  3. Open self DM
  4. Tap plus icon next to chat input

Expected Result:

In self DM, track expense must be shown .

Actual Result:

In self DM, create expense is shown instead track expense.

Workaround:

Unknown

Platforms:

Screenshots/Videos

https://github.com/user-attachments/assets/22563c18-62fc-447d-a2f4-0b190bcdb3c5

View all open jobs on GitHub

melvin-bot[bot] commented 17 hours ago

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

mkzie2 commented 17 hours ago

Edited by proposal-police: This proposal was edited at 2024-11-21 13:15:52 UTC.

Proposal

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

In self DM, create expense is shown instead track expense.

What is the root cause of that problem?

We display Create expense if canUseCombinedTrackSubmit is true.

https://github.com/Expensify/App/blob/461781621a22903b64534202c95811890bf05df4/src/pages/home/report/ReportActionCompose/AttachmentPickerWithMenuItems.tsx#L162-L163

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

I think we should display Track expense for all case in the plus icon of SelfDM

icon:  getIconForAction(CONST.IOU.TYPE.TRACK),
text: translate('iou.trackExpense'),

https://github.com/Expensify/App/blob/461781621a22903b64534202c95811890bf05df4/src/pages/home/report/ReportActionCompose/AttachmentPickerWithMenuItems.tsx#L162-L163

OPTIONAL: We can also change this to always display Track expense

https://github.com/Expensify/App/blob/461781621a22903b64534202c95811890bf05df4/src/pages/iou/request/IOURequestStartPage.tsx#L58

What alternative solutions did you explore? (Optional)

NA