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

[$250] Web - Trip - Payment with Expensify button shown next to submit button on Open trip report #54538

Open
2 of 8 tasks
IuliiaHerets opened this issue Dec 24, 2024 · 31 comments
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. External Added to denote the issue can be worked on by a contributor Reviewing Has a PR in review Weekly KSv2

Comments

@IuliiaHerets
Copy link

IuliiaHerets commented Dec 24, 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.78-1
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail: #52823
Email or phone of affected tester (no customers): applausetester+0948501@applause.expensifail.com
Issue reported by: Applause Internal Team
Device used: Windows 11 / Chrome
App Component: Chat Report View

Action Performed:

Precondition:

  1. Go to staging.new.expensify.com
  2. Go to trip room.
  3. Click on the expense preview.

Expected Result:

There should only be a Submit button at the top of the report page.

Actual Result:

There are two buttons - "Pay with Expensify" and "Submit"

Workaround:

Unknown

Platforms:

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

Screenshots/Videos

Bug6701759_1735043098180.bandicam_2024-12-24_20-22-00-193.mp4

Bug6701759_1735043098161!bandicam_2024-12-24_20-22-09-016

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~021876334627155535758
  • Upwork Job ID: 1876334627155535758
  • Last Price Increase: 2025-01-06
  • Automatic offers:
    • shubham1206agra | Contributor | 105583255
@IuliiaHerets IuliiaHerets added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Dec 24, 2024
Copy link

melvin-bot bot commented Dec 24, 2024

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.

@sakluger
Copy link
Contributor

The actual bug here is that the expense report shows a Pay with expensify button at all. The report is still open, so it shouldn't have a Pay option, in fact, I can't think of any scenario where we would show both a submit and pay button at the same time.

@rojiphil @jacobkim9881 do you think this is related to #53226 at all?

@sakluger sakluger changed the title Web - Trip - No gap between payment button and submit button in trip report Web - Trip - Payment with Expensify button shown next to submit button on Open trip report Dec 24, 2024
@jacobkim9881
Copy link
Contributor

@sakluger I not sure if it is related. I have tried Action Performed but I couldn't because I can't enable travel. I can test if I get a company email.
스크린샷, 2024-12-27 10-59-05

@melvin-bot melvin-bot bot added the Overdue label Dec 27, 2024
Copy link

melvin-bot bot commented Dec 30, 2024

@sakluger Huh... This is 4 days overdue. Who can take care of this?

@sakluger
Copy link
Contributor

@jacobkim9881 we can't get you a company email, but I can add your testing email to the travel beta. What email are you using to test?

@melvin-bot melvin-bot bot removed the Overdue label Dec 30, 2024
@sakluger
Copy link
Contributor

Actually, it looks like you would have to actually book travel to test, which would require you to spend real money. I'm going to ask around internally first.

@jacobkim9881
Copy link
Contributor

Actually, it looks like you would have to actually book travel to test, which would require you to spend real money. I'm going to ask around internally first.

I understood. I will wait for internal team.

@jacobkim9881
Copy link
Contributor

jacobkim9881 commented Dec 30, 2024

we can't get you a company email, but I can add your testing email to the travel beta. What email are you using to test?

I know I should actually book travel to test but in case my email is like below:

@jacobkim9881

This comment was marked as resolved.

@melvin-bot melvin-bot bot added the Overdue label Jan 2, 2025
@sakluger
Copy link
Contributor

sakluger commented Jan 2, 2025

I asked for help in Slack here to see if this is external or internal: https://expensify.slack.com/archives/C05S5EV2JTX/p1735839407611239

@sakluger
Copy link
Contributor

sakluger commented Jan 3, 2025

Hey @jacobkim9881 - I set jacobkim9881+173@gmail.com as a Travel test account. Please go to your account settings in Expensify Classic and add a first and last name.

Then, to book fake travel:

  1. Go to Travel in the LHN
  2. Click the "Book or manage Travel" button
  3. Check the checkbox to accept terms
  4. Enter an address (it can be fake)
  5. Click confirm and wait
  6. You will then be logged into Spotnana

Then, Inside of Spotnana:

  1. Search for a new booking
    1. For flights:
      • This can be flaky, so it is recommended to use the following flight
      • One way from SEA to SFO
      • Alaska Airlines
      • At least 1 month in the future
    2. For hotels:
      1. Search for any major airport (usually SFO, NYC, LHR)
      2. Book any of top chains (usually Marriott, Hilton)
  2. Select the flight
  3. Add the other required information and click save at the top of the panel
  4. Click "next" on the right side
  5. Click "select a trip" and create a new trip
  6. Add the stripe test card ending in 4444. All data such as address, name etc. can be fake, as long as the expiration is after the current date
  7. Book the trip

@sakluger
Copy link
Contributor

sakluger commented Jan 3, 2025

Once you book the fake travel, you should see the trip report in NewDot.

@jacobkim9881
Copy link
Contributor

@sakluger I understood and let me test these step.

@jacobkim9881
Copy link
Contributor

@sakluger Thanks for guiding me with the steps. I have added a first and last name at account setting in OD. I couldn't find "Book or manage Travel" button at Travel in LHN of OD but I have this screen.
스크린샷, 2025-01-03 11-12-58

@sakluger
Copy link
Contributor

sakluger commented Jan 3, 2025

Sorry @jacobkim9881 - I forgot to add you to the Travel beta. I've added you now, and so the button should show up within the next hour (betas can take up to 1 hour to be applied). Let me know if you run into any other issues.

@jacobkim9881
Copy link
Contributor

@sakluger Indeed, there occurs enable button. It still needs company email account though. I hope I got to know any workaround to enable.
스크린샷 2025-01-04 오전 4 07 59

@shubham1206agra
Copy link
Contributor

@sakluger It will be easier if you assign this to me and @blazejkustra since we have trip rooms present.

@melvin-bot melvin-bot bot added the Overdue label Jan 6, 2025
@sakluger
Copy link
Contributor

sakluger commented Jan 6, 2025

@jacobkim9881 thanks for trying! Sorry you weren't able to get it working without a company email.

@shubham1206agra and @blazejkustra I will assign you two as suggested.

@melvin-bot melvin-bot bot removed the Overdue label Jan 6, 2025
@sakluger sakluger added the External Added to denote the issue can be worked on by a contributor label Jan 6, 2025
@melvin-bot melvin-bot bot removed the Help Wanted Apply this label when an issue is open to proposals by contributors label Jan 6, 2025
Copy link

melvin-bot bot commented Jan 6, 2025

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

@melvin-bot melvin-bot bot removed the Overdue label Jan 6, 2025
Copy link

melvin-bot bot commented Jan 6, 2025

📣 @shubham1206agra 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app!

Offer link
Upwork job
Please accept the offer and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑‍💻
Keep in mind: Code of Conduct | Contributing 📖

@sakluger
Copy link
Contributor

sakluger commented Jan 6, 2025

@blazejkustra could you please comment on the issue so that I can assign you?

Copy link

melvin-bot bot commented Jan 7, 2025

@sakluger @shubham1206agra this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

@blazejkustra
Copy link
Contributor

Sorry, I was OOO until today. Commenting for assigment @sakluger

@blazejkustra
Copy link
Contributor

blazejkustra commented Jan 7, 2025

@IuliiaHerets I can't reproduce it, all my trip previews don't have 'Submit' button, maybe your workspace is configured in another way 🤔 Could you add me to that workspace and make me an admin? Here is my email: blazej.kustra+trav3@swmansion.com

Before proceeding to implementation, I'd like to confirm the expected behaviour here:

Expected Result:

There should only be a Submit button at the top of the report page.

Actual Result:

There are two buttons - "Pay with Expensify" and "Submit"

@kavimuru
Copy link

kavimuru commented Jan 7, 2025

Added you as a admin @blazejkustra

@stitesExpensify
Copy link
Contributor

@sakluger can you confirm the expected behavior here? #54538 (comment)

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

Yes, since this is an open (or draft) report, it should only have a submit button. The report submitter should never see a pay button, and I can't think of any scenario where an open report would have a pay button or that the Submit and Pay buttons would show up at the same time.

@stitesExpensify do you think that the expected behavior is different?

@melvin-bot melvin-bot bot removed the Overdue label Jan 10, 2025
@stitesExpensify
Copy link
Contributor

Nope, that makes sense to me! I just wanted to make sure we were pointing @blazejkustra in the right direction

@sakluger
Copy link
Contributor

Got it, sounds good!

Copy link

melvin-bot bot commented Jan 15, 2025

Triggered auto assignment to @blimpich, see https://stackoverflow.com/c/expensify/questions/7972 for more details.

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. External Added to denote the issue can be worked on by a contributor Reviewing Has a PR in review Weekly KSv2
Development

No branches or pull requests

9 participants