App icon indicating copy to clipboard operation
App copied to clipboard

[$500] Send money -For send money, merchant shown as " Request "

Open lanitochka17 opened this issue 1 year ago • 9 comments

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: 1.4.36 Reproducible in staging?: Y Reproducible in production?: Y If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4281379 Email or phone of affected tester (no customers): Logs: https://stackoverflow.com/c/expensify/questions/4856 Expensify/Expensify Issue URL: Issue reported by: Applause - Internal Team Slack conversation:

Action Performed:

  1. Launch app
  2. Tap fab----Send money
  3. Enter an amount and tap next
  4. Select a contact
  5. Tap payelsewhere
  6. Tap the send money request created to open details page
  7. Note the merchant field

Expected Result:

For send money, merchant must not be shown as " Request "

Actual Result:

For send money, merchant shown as " Request "

Workaround:

Unknown

Platforms:

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

  • [x] Android: Native
  • [x] Android: mWeb Chrome
  • [x] iOS: Native
  • [x] iOS: mWeb Safari
  • [x] MacOS: Chrome / Safari
  • [x] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/2e12ba07-b1c8-4392-9a0b-1135e28a4325

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~0133847fab45efc929
  • Upwork Job ID: 1754204504573894656
  • Last Price Increase: 2024-02-04

lanitochka17 avatar Feb 04 '24 18:02 lanitochka17

Job added to Upwork: https://www.upwork.com/jobs/~0133847fab45efc929

melvin-bot[bot] avatar Feb 04 '24 18:02 melvin-bot[bot]

Triggered auto assignment to Contributor-plus team member for initial proposal review - @alitoshmatov (External)

melvin-bot[bot] avatar Feb 04 '24 18:02 melvin-bot[bot]

Triggered auto assignment to @CortneyOfstad (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

melvin-bot[bot] avatar Feb 04 '24 18:02 melvin-bot[bot]

It seems this Bug is from BE

DylanDylann avatar Feb 04 '24 18:02 DylanDylann

Current assignee @alitoshmatov is eligible for the Internal assigner, not assigning anyone new.

melvin-bot[bot] avatar Feb 05 '24 19:02 melvin-bot[bot]

Seems related to: https://github.com/Expensify/App/pull/34216

muttmuure avatar Feb 05 '24 19:02 muttmuure

What is the expectation, that the merchant line is not shown at all, or that something other than "Request" is shown for the merchant? Also, what makes this a backend issue?

tgolen avatar Feb 05 '24 23:02 tgolen

That's a good question… If a user is just sending money to another user, does there need to be a merchant line at all? Wouldn't the merchant just be the sender? Maybe we should require the description field for Send Money to be filled out similar to Venmo.

deetergp avatar Feb 05 '24 23:02 deetergp

The merchant value of Request is coming from the back end. Screenshot 2024-02-05 at 3 39 42 PM Screenshot 2024-02-05 at 3 37 20 PM

deetergp avatar Feb 05 '24 23:02 deetergp

It's set in Transaction::createMoneyRequest in Auth.

deetergp avatar Feb 06 '24 01:02 deetergp

There is already a nail for "Request" (CONST.TRANSACTION.DEFAULT_MERCHANT) in the MoneyRequestPreview. https://github.com/Expensify/App/blob/500dd0949123ae1cd7b84211c0093f3c832d6775/src/components/ReportActionItem/MoneyRequestPreview.tsx#L163

lisishyn avatar Feb 06 '24 09:02 lisishyn

📣 @lisishyn! 📣 Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork. Please follow these steps:

  1. Make sure you've read and understood the contributing guidelines.
  2. Get the email address used to login to your Expensify account. If you don't already have an Expensify account, create one here. If you have multiple accounts (e.g. one for testing), please use your main account email.
  3. Get the link to your Upwork profile. It's necessary because we only pay via Upwork. You can access it by logging in, and then clicking on your name. It'll look like this. If you don't already have an account, sign up for one here.
  4. Copy the format below and paste it in a comment on this issue. Replace the placeholder text with your actual details. Screen Shot 2022-11-16 at 4 42 54 PM Format:
Contributor details
Your Expensify account email: <REPLACE EMAIL HERE>
Upwork Profile Link: <REPLACE LINK HERE>

melvin-bot[bot] avatar Feb 06 '24 09:02 melvin-bot[bot]

Contributor details Your Expensify account email: [email protected] Upwork Profile Link: https://www.upwork.com/freelancers/~015703306bbdfe0891

lisishyn avatar Feb 06 '24 09:02 lisishyn

✅ Contributor details stored successfully. Thank you for contributing to Expensify!

melvin-bot[bot] avatar Feb 06 '24 09:02 melvin-bot[bot]

Started an internal conversation about this in #vip-split.

deetergp avatar Feb 06 '24 18:02 deetergp

Based on the conversation in the above thread, we are ultimately going to only ever show Amount and Description on Send money requests. Merchant will no longer be display once the work in this internal GH is complete. Closing this one out.

deetergp avatar Feb 06 '24 20:02 deetergp