[$500] SplitBill date is incorrect when a date other than today is selected

[m-natarajan]

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!

---

Version Number: 1.4.39-0
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: @tgolen
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1707500020195589

Action Performed:

1. Create a split bill
2. On the confirmation screen, set the date to a date in the future
3. After creating the bill, click on the preview to view the details

Expected Result:

The date should show as date that is not today

Actual Result:

The date shows today's date because it's showing the "created" date and not the date the user specified.

Workaround:

unknown

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

Add any screenshot/video evidence

Recording.2713.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~01d790dc8609cd540a
• Upwork Job ID: 1757674654685200384
• Last Price Increase: 2024-02-14

[melvin-bot]

Triggered auto assignment to @stephanieelliott (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

[vishalsanghai]

Should we allow entries with future date?
If not we can disable selecting any future date on UI.

[melvin-bot]

📣 @vishalsanghai! 📣
Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork.
Please follow these steps:

1. Make sure you've read and understood the contributing guidelines.
2. Get the email address used to login to your Expensify account. If you don't already have an Expensify account, create one here. If you have multiple accounts (e.g. one for testing), please use your main account email.
3. Get the link to your Upwork profile. It's necessary because we only pay via Upwork. You can access it by logging in, and then clicking on your name. It'll look like this. If you don't already have an account, sign up for one here.
4. Copy the format below and paste it in a comment on this issue. Replace the placeholder text with your actual details.
   
   Format:

Contributor details
Your Expensify account email: <REPLACE EMAIL HERE>
Upwork Profile Link: <REPLACE LINK HERE>

[vishalsanghai]

Contributor details
Your Expensify account email: [email protected]
Upwork Profile Link: https://www.upwork.com/freelancers/~0121fccdaccc2abdc1

[melvin-bot]

✅ Contributor details stored successfully. Thank you for contributing to Expensify!

[brunovjk]

Interested in help, despite the absence of the "Help Wanted" tag. Noticed this issue while working on another PR and @DylanDylann helped with the solution. However, I believe we still need an internal team member to check and implement the required backend adjustments.

[DylanDylann]

Could I take this issue as C+ contributor because I have context and discuss here

[tgolen]

Should we allow entries with future date?

I noticed this question and wanted to respond. This should be allowed, yes. Please use these constants for the date.

[tgolen]

Assigned to you @DylanDylann to work on this.

[melvin-bot]

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

[melvin-bot]

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

[stephanieelliott]

Seems like the labels didn't apply correctly here. Thanks for noticing @brunovjk -- giving you a heads up if you want to make a proposal 😊

[jonxzsh]

It seems like "created" is passed to the RequestMoney command with a value similar to "2024-02-17", but the same doesn't happen on SplitBill. I don't have my environment setup so just posting to anybody who's looking. Seems like you might have to make API changes aswell to implement the "created" parameter and do something with it on the server.

[melvin-bot]

📣 @jonsystems! 📣
Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork.
Please follow these steps:

1. Make sure you've read and understood the contributing guidelines.
2. Get the email address used to login to your Expensify account. If you don't already have an Expensify account, create one here. If you have multiple accounts (e.g. one for testing), please use your main account email.
3. Get the link to your Upwork profile. It's necessary because we only pay via Upwork. You can access it by logging in, and then clicking on your name. It'll look like this. If you don't already have an account, sign up for one here.
4. Copy the format below and paste it in a comment on this issue. Replace the placeholder text with your actual details.
   
   Format:

Contributor details
Your Expensify account email: <REPLACE EMAIL HERE>
Upwork Profile Link: <REPLACE LINK HERE>

[brunovjk]

Proposal

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

SplitBill date displays incorrectly when a date other than today is selected.

What is the root cause of that problem?

Missing created parameters at SplitBillParams.

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

• Add created parameters at SplitBillParams.
• Update IOURequestStepConfirmation to pass the created transaction value to IOU.splitBill and IOU.splitBillAndOpenReport call.
• Update splitBill, splitBillAndOpenReport and createSplitsAndOnyxData to receive and use created value to buildOptimisticTransaction.

Note

With these changes, transaction.created is correct, but we must also update BE to use the created parameter when we call SPLIT_BILL and SPLIT_BILL_AND_OPEN_REPORT, as we do in COMPLETE_SPLIT_BILL, which returns the correct date.

POC: Create a splitBillAndOpenReport offline and than go online:

Proposed changes

POC-36285-Proposed.changes.mp4

Latest main

POC-36285-Latest.main.mp4

What alternative solutions did you explore? (Optional)

[DylanDylann]

@brunovjk Your proposal looks good to me

Note that: We need to update splitBillAndOpenReport API and splitBill API from the BE side to fix this issue completely. Don't need to update StartSplitBill API, because this flow is used when splitting bill with receipt and date field is hidden in this case

🎀 👀 🎀 C+ reviewed

[melvin-bot]

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

[brunovjk]

Don't need to update StartSplitBill API

Exactly!
Thanks for the review @DylanDylann :D

[francoisl]

Hi @brunovjk, looks like the backend was just modified to support the created param for the commands SplitBill and SplitBillAndOpenReport a few days ago. Can you try your changes again and see if the backend returns the right date in the response now?

[francoisl]

Ah shoot, actually this might be the same issue as #34526, which just had this fix merged: #35141. Right?

[DylanDylann]

Thank you for pointing out that. Can't reproduce anymore

[stephanieelliott]

I can't repro it anymore either -- gonna close this as resolved.