[HOLD for payment 2024-08-19] [P2P Distance] Enable split distance requests

[neil-marcellini]

Enable split distance requests following this plan

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~017fd731ba8df719c5
• Upwork Job ID: 1760074110878371840
• Last Price Increase: 2024-02-20

[melvin-bot]

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

[melvin-bot]

Triggered auto assignment to Contributor Plus for review of internal employee PR - @jjcoffee (Internal)

[melvin-bot]

@jjcoffee Eep! 4 days overdue now. Issues have feelings too...

[melvin-bot]

@jjcoffee Still overdue 6 days?! Let's take care of this!

[jjcoffee]

Chill, Melv ✌️

[melvin-bot]

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

[neil-marcellini]

I'm starting on this now. I will split up the plan into multiple PRs to keep each one small and focused. I will start with moving split participant normalization to Auth.

Actually, now that I look more closely, it makes sense to move the creation of an individual payer account to Auth first. Then we can re-use that Auth function to create all split accounts as needed. Therefore I'll HOLD this issue on the issue to implement that part [P2P Distance] Move creating the payer account for money requests from Web-E to Auth

[neil-marcellini]

On hold for a bit, moving to weekly

[neil-marcellini]

On hold

[neil-marcellini]

I got all the PRs up for review. 🎉 It was a lot of work and there will be some more to do 😅
[Auth] [HOLD Web-E 42700] Enable distance splits
[Web-Expensify] [HOLD Auth 10851] Enable splits for distance expenses
[App] [HOLD Web-E 41977] Enable distance splits

[melvin-bot]

This issue has not been updated in over 15 days. @neil-marcellini eroding to Monthly issue.

P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do!

[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.

[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.

[melvin-bot]

Reviewing label has been removed, please complete the "BugZero Checklist".

[melvin-bot]

The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.18-10 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

• Enable distance splits #42302

If no regressions arise, payment will be issued on 2024-08-19. 🎊

[neil-marcellini]

Head's up, I'll be working 50% for the rest of this week and next 🌲

[melvin-bot]

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

[neil-marcellini]

@twisterdotcom would you please handle payment for the C+ @mananjadhav

[twisterdotcom]

@neil-marcellini what are we pricing this at? Normal $250?

[twisterdotcom]

Going to go with $250 as normal here.

Payment Summary:

• @mananjadhav paid $250 via newDot requests (C+)

[JmillsExpensify]

$250 approved for @mananjadhav