[HOLD for payment 2024-12-30] [$250] Refactor createDistanceRequest to use a parameter object

[neil-marcellini]

As part of the tracking issue, and as advised in its description, refactor createDistanceRequest to use a parameter object.

cc @mkzie2 @dukenv0307

Issue Owner

Current Issue Owner: @

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~021866886683506920885
• Upwork Job ID: 1866886683506920885
• Last Price Increase: 2024-12-11

Issue Owner

Current Issue Owner: @OfstadC

[mkzie2]

Proposal

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

Refactor createDistanceRequest to use a parameter

What is the root cause of that problem?

This is an improvement

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

Wrap all parameters of this function into one object and in this object, we can have some sub-objects that will wrap the related data.

App/src/libs/actions/IOU.ts

Line 5310 in 4f4720e

function createDistanceRequest(

What specific scenarios should we cover in automated tests to prevent reintroducing this issue in the future?

What alternative solutions did you explore? (Optional)

[mkzie2]

@neil-marcellini I'm here.

[dukenv0307]

I can take it as C+ based on this comment

[mkzie2]

@neil-marcellini Is this ready?

[melvin-bot]

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

[melvin-bot]

Current assignee @dukenv0307 is eligible for the External assigner, not assigning anyone new.