CSV import does not handle date formats without a delimiter #107

Closed
opened 2026-02-28 18:48:43 -06:00 by GiteaMirror · 2 comments
Owner

Originally created by @TomAFrench on GitHub (Jul 2, 2022).

I was trying to import a CSV file earlier and discovered that we currently can't parse the dates as it's just concatenated with no delimiter.

image

We should support parsing dates of this type

Originally created by @TomAFrench on GitHub (Jul 2, 2022). I was trying to import a CSV file earlier and discovered that we currently can't parse the dates as it's just concatenated with no delimiter. ![image](https://user-images.githubusercontent.com/15848336/177010882-dd78d469-7cf7-4bf6-b65a-0db244ffe993.png) We should support parsing dates of this type
Author
Owner

@PartyLich commented on GitHub (Jul 2, 2022):

Are you already working on this? If not, I'm interested.
eg
image

The relevant bits are in loot-design and either don't have tests or I just didn't spot them. In the latter case, a pointer in the right direction would be grand.

@PartyLich commented on GitHub (Jul 2, 2022): Are you already working on this? If not, I'm interested. eg ![image](https://user-images.githubusercontent.com/2792750/177018861-b26aad5e-98e3-49f6-afe4-84852f3ccb54.png) The relevant bits are in loot-design and either don't have tests or I just didn't spot them. In the latter case, a pointer in the right direction would be grand.
Author
Owner

@TomAFrench commented on GitHub (Jul 3, 2022):

Are you already working on this? If not, I'm interested.

Feel free to go ahead. I only get this on an account which has very few transactions so it's not a big priority for me (I can just manually input them for now). I'll probably get round to it at some point.

Looks like you'd need to make a couple of modifications to ImportTransactions.js. I can't see any tests of this functionality as it is either.

@TomAFrench commented on GitHub (Jul 3, 2022): > Are you already working on this? If not, I'm interested. Feel free to go ahead. I only get this on an account which has very few transactions so it's not a big priority for me (I can just manually input them for now). I'll probably get round to it at some point. Looks like you'd need to make a couple of modifications to `ImportTransactions.js`. I can't see any tests of this functionality as it is either.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#107