[Awaiting Payment Sept 24th] [$125] Pay elsewhere missing as a payment method when configured for direct reimbursement on a workspace

[trjExpensify]

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.29-7
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @trjExpensify
Slack conversation: #36301 (comment)

Action Performed:

1. Create a workspace
2. Go to More features > Enable Workflows
3. Go to Workflows > Enable approvals and payments
4. Click Connect bank account and add a VBBA
5. Submit an expense report on the workspace
6. Go to the expense report
7. Observe the Pay button

Expected Result:

The pay button should have an option to change it to Pay elsewhere to mark the report as manually reimbursed.

Actual Result:

The pay button does not have that option, only Pay $x using Expensify.

Workaround:

No

Platforms:

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

• Android: Native
• Android: mWeb Chrome
• iOS: Native
• iOS: mWeb Safari
• MacOS: Chrome / Safari
• MacOS: Desktop

Screenshots/Videos

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~021831725635476300859
• Upwork Job ID: 1831725635476300859
• Last Price Increase: 2024-09-05
• Automatic offers:
• shubham1206agra | Contributor | 103842728

Issue Owner

Current Issue Owner: @trjExpensify

[melvin-bot]

Current assignee @trjExpensify is eligible for the Bug assigner, not assigning anyone new.

[melvin-bot]

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

[melvin-bot]

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

[melvin-bot]

Upwork job price has been updated to $125

[nyomanjyotisa]

Proposal

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

Pay elsewhere missing as a payment method when configured for direct reimbursement on a workspace

What is the root cause of that problem?

The shouldShowPayElsewhereOption here is return false because both !isPaidGroupPolicy and policy?.reimbursementChoice === CONST.POLICY.REIMBURSEMENT_CHOICES.REIMBURSEMENT_MANUAL are false

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

we displaying the VBBA option if both isExpenseReport and shouldShowPaywithExpensifyOption are true

So add another condition on shouldShowPayElsewhereOption to show it if both isExpenseReport and shouldShowPaywithExpensifyOption are true

const shouldShowPayElsewhereOption = (!isPaidGroupPolicy || policy?.reimbursementChoice === CONST.POLICY.REIMBURSEMENT_CHOICES.REIMBURSEMENT_MANUAL || (isExpenseReport && shouldShowPaywithExpensifyOption)) && !isInvoiceReport;

also we need to move the shouldShowPayElsewhereOption to inside paymentButtonOptions after isExpenseReport declaration

Result

What alternative solutions did you explore? (Optional)

[neonbhai]

Proposal

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

Pay elsewhere missing as a payment method when configured for direct reimbursement on a workspace

What is the root cause of that problem?

We only allow the Pay Elsewhere option when policy.reimbursementChoice is REIMBURSEMENT_MANUAL.

We now want to show the option when reimbursement choice is either Manual or REIMBURSEMENT_YES.

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

We will change the logic here:

App/src/components/SettlementButton.tsx

Line 171 in e7bd29a

const shouldShowPayElsewhereOption = (!isPaidGroupPolicy || policy?.reimbursementChoice === CONST.POLICY.REIMBURSEMENT_CHOICES.REIMBURSEMENT_MANUAL) && !isInvoiceReport;

to:

const shouldShowPayElsewhereOption = (!isPaidGroupPolicy || policy?.reimbursementChoice 
    	!== CONST.POLICY.REIMBURSEMENT_CHOICES.REIMBURSEMENT_NO) && !isInvoiceReport;

[Nodebrute]

Proposal

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

Pay elsewhere missing as a payment method when configured for direct reimbursement on a workspace

What is the root cause of that problem?

Here we only show pay elsewhere only when policy?.reimbursementChoice is reimburseManual but when bank account is accounted the policy?.reimbursementChoice is reimburseYes

App/src/components/SettlementButton.tsx

Line 171 in cc1c7d0

const shouldShowPayElsewhereOption = (!isPaidGroupPolicy || policy?.reimbursementChoice === CONST.POLICY.REIMBURSEMENT_CHOICES.REIMBURSEMENT_MANUAL) && !isInvoiceReport;

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

We don't need the policy?.reimbursementChoice check because in canIOUBePaid we already hide the button when policy?.reimbursementChoice === reimburseNo and for other two options we want to show pay elsewhere button

App/src/libs/actions/IOU.ts

Lines 6844 to 6846 in 8fc5e68

	if (policy?.reimbursementChoice === CONST.POLICY.REIMBURSEMENT_CHOICES.REIMBURSEMENT_NO) {
	return false;
	}

We can change this code to

  const shouldShowPayElsewhereOption = !isPaidGroupPolicy || !isInvoiceReport;

What alternative solutions did you explore? (Optional)

Or we can change this to

  const shouldShowPayElsewhereOption =  !isInvoiceReport;

[melvin-bot]

📣 @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 📖

[mountiny]

We chose @shubham1206agra to work on it based on internal slack discussion and clarification of the logic. Apologies for not making this cleaner before and losing some time then @shubham1206agra What is your ETA for a PR

[melvin-bot]

@trjExpensify, @mountiny, @shubham1206agra, @aimane-chnaif Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

[trjExpensify]

@shubham1206agra didn't you have a PR for this?

[melvin-bot]

⚠️ Looks like this issue was linked to a Deploy Blocker here

If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results.

If a regression has occurred and you are the assigned CM follow the instructions here.

If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.

[trjExpensify]

Payment will be due tomorrow. 👍

[trjExpensify]

Payment summary as follows:

• $125 to @shubham1206agra for the fix
• $125 to @aimane-chnaif for the review

Aimane, go ahead and request. @shubham1206agra please accept the offer, it's waiting on yah.

[trjExpensify]

and Melvin, please feel free to jump in a lake.

[trjExpensify]

@shubham1206agra can you accept the offer so we can close this?

[shubham1206agra]

@trjExpensify Accepted offer

[aimane-chnaif]

I am still using upwork

[trjExpensify]

I am still using upwork

Oh really? Why so? You're in the list for NewDot. 🤔

[aimane-chnaif]

yes I am eligible but I moved back to upwork

[trjExpensify]

Got it. @mallenexpensify will work on getting that update. In the meantime, an offer has been sent.

[melvin-bot]

@trjExpensify @mountiny @shubham1206agra @aimane-chnaif this issue is now 4 weeks old, please consider:

• Finding a contributor to fix the bug
• Closing the issue if BZ has been unable to add the issue to a VIP or Wave project
• If you have any questions, don't hesitate to start a discussion in #expensify-open-source

Thanks!

[trjExpensify]

Paid, closing!

[aimane-chnaif]

@trjExpensify Payment failed due to upwork bug. Can you please reopen?

[trjExpensify]

I've paid a bonus to get around that. Let me know if that doesn't work.

[aimane-chnaif]

Received bonus. Thanks