-
Notifications
You must be signed in to change notification settings - Fork 3.9k
[NO QA][Odometer] Create NewDot Odometer expense flow #78919
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
Changes from all commits
8eda82c
139eaf6
9ad8cfa
f91e66a
95a006e
add4518
fb26c40
b53561b
2007e56
a5c0fde
2c98761
01bca21
61951eb
581d8f1
fc7ca57
612f908
295e5b1
6354c70
7d3cff2
28553f5
401d39a
f56c72d
a2541d1
9cdeffb
deb334b
1c682ec
bdf3c4e
9498e77
ddc1d1c
052ebca
2aa27e3
03f3bf5
5389ed7
a387ca2
ca6c090
d4f4e20
fe7f38a
a1f0a3a
25d2fc3
c7dddf6
4b43d42
9421c57
b732727
e364b82
e07fc17
0869336
b4ae629
79e8c7b
15aa760
7add61d
90ca4ea
d55f79d
86e8c6c
1d8b577
19728d1
41282e8
5ea58ee
286db57
9daf5a3
e99e8af
2d0cd84
5368398
6a3173b
e0acf4c
d9ade69
5b71f11
cabc1fd
9d861a6
7fb8488
e2837ff
5c7da37
4a7515e
fc48efb
56d82ef
8a4bcf5
817b517
a155760
bd786d6
fe2a52f
53e82e4
4a1e38c
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||||
|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -1175,6 +1175,17 @@ const ROUTES = { | |||||||||
| return getUrlWithBackToParam(`${action as string}/${iouType as string}/distance-manual/${transactionID}/${reportID}${reportActionID ? `/${reportActionID}` : ''}`, backTo); | ||||||||||
| }, | ||||||||||
| }, | ||||||||||
| MONEY_REQUEST_STEP_DISTANCE_ODOMETER: { | ||||||||||
| route: ':action/:iouType/distance-odometer/:transactionID/:reportID', | ||||||||||
| getRoute: (action: IOUAction, iouType: IOUType, transactionID: string | undefined, reportID: string | undefined, backTo = '') => { | ||||||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @Julesssss we shouldn't use backTo param in new implementation following the guideline App/contributingGuides/NAVIGATION.md Lines 438 to 441 in f330976
I think we should create another issue to remove the backTo param to avoid block the main implementaion
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Completely agree, being handled here |
||||||||||
| if (!transactionID || !reportID) { | ||||||||||
| Log.warn('Invalid transactionID or reportID is used to build the MONEY_REQUEST_STEP_DISTANCE_ODOMETER route'); | ||||||||||
| } | ||||||||||
|
|
||||||||||
| // eslint-disable-next-line no-restricted-syntax -- Legacy route generation | ||||||||||
| return getUrlWithBackToParam(`${action as string}/${iouType as string}/distance-odometer/${transactionID}/${reportID}`, backTo); | ||||||||||
| }, | ||||||||||
| }, | ||||||||||
| MONEY_REQUEST_STEP_DISTANCE_RATE: { | ||||||||||
| route: ':action/:iouType/distanceRate/:transactionID/:reportID/:reportActionID?', | ||||||||||
| getRoute: (action: IOUAction, iouType: IOUType, transactionID: string | undefined, reportID: string | undefined, backTo = '', reportActionID?: string) => { | ||||||||||
|
|
@@ -1294,6 +1305,11 @@ const ROUTES = { | |||||||||
| getRoute: (action: IOUAction, iouType: IOUType, transactionID: string | undefined, reportID: string | undefined, backToReport?: string) => | ||||||||||
| `${action as string}/${iouType as string}/start/${transactionID}/${reportID}/distance-new${backToReport ? `/${backToReport}` : ''}/distance-gps` as const, | ||||||||||
| }, | ||||||||||
| DISTANCE_REQUEST_CREATE_TAB_ODOMETER: { | ||||||||||
| route: 'distance-odometer', | ||||||||||
| getRoute: (action: IOUAction, iouType: IOUType, transactionID: string, reportID: string, backToReport?: string) => | ||||||||||
| `${action as string}/${iouType as string}/start/${transactionID}/${reportID}/distance-new${backToReport ? `/${backToReport}` : ''}/distance-odometer` as const, | ||||||||||
| }, | ||||||||||
| IOU_SEND_ADD_BANK_ACCOUNT: 'pay/new/add-bank-account', | ||||||||||
| IOU_SEND_ADD_DEBIT_CARD: 'pay/new/add-debit-card', | ||||||||||
| IOU_SEND_ENABLE_PAYMENTS: 'pay/new/enable-payments', | ||||||||||
|
|
||||||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Julesssss we shouldn't use backTo param in new implementation following the guideline
App/contributingGuides/NAVIGATION.md
Lines 438 to 441 in f330976
I think we should create another issue to remove the backTo param to avoid block the main implementaion
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Completely agree, will will handle this here.