Skip to content
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

[Bug]: Date parser in csv import confuses between two formats #2310

Closed
3 of 5 tasks
ishanjain28 opened this issue Feb 1, 2024 · 2 comments
Closed
3 of 5 tasks

[Bug]: Date parser in csv import confuses between two formats #2310

ishanjain28 opened this issue Feb 1, 2024 · 2 comments
Labels
bug Something isn't working

Comments

@ishanjain28
Copy link

Verified issue does not already exist?

  • I have searched and found no existing issue

Is this related to GoCardless, Simplefin or another bank-sync provider?

  • I have checked my server logs and could not see any errors there
  • I will be attaching my server logs to this issue
  • I will be attaching my client-side (browser) logs to this issue
  • I understand that this issue will be automatically closed if insufficient information is provided

What happened?

My bank generates CSV files which use DD/MM/YY format. I have set Date format to DD/MM/YY in AB and it is still parsing them as MM/DD/YY.

image

A redacted csv file that can be used to replicate this bug.


  Date
 01/01/24
 03/01/24
 10/01/24
 19/01/24
 19/01/24
 21/01/24
 01/02/24
 01/02/24
 01/02/24

What error did you receive?

There are no errors, It parses the input incorrectly.

Where are you hosting Actual?

Locally via Yarn

What browsers are you seeing the problem on?

Firefox

Operating System

Linux

@ishanjain28 ishanjain28 added the bug Something isn't working label Feb 1, 2024
@pprimor
Copy link
Contributor

pprimor commented Feb 2, 2024

I'm not sure what the issue is here. Did you try importing the transactions to see if the dates were, in fact, wrongly parsed?

Keep in mind that, in the screenshot you posted, the parsed dates are in the format MM/DD/YYYY. This is the default for the app but you can change it anytime in the settings.

Screenshot 2024-02-02 at 10 21 38

@ishanjain28
Copy link
Author

Thank you @pprimor That was it!

I didn't realize I had to change date to dd/mm/yy in more -> settings.

It works perfectly after changing date in settings.
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants