Browse Source
Feature/enforce yyyy instead of yy as date format in activities import (#2036)
* Enforce yyyy (instead of yy)
* Update changelog
pull/2037/head
Thomas Kaul
2 years ago
committed by
GitHub
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with
12 additions and
3 deletions
-
CHANGELOG.md
-
apps/client/src/app/services/import-activities.service.ts
-
test/import/invalid-date-yy.csv
|
|
@ -14,6 +14,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 |
|
|
|
### Changed |
|
|
|
|
|
|
|
- Improved the routes of the tabs |
|
|
|
- Enforced a stricter date format in the activities import: `dd-MM-yyyy` instead of `dd-MM-yy` |
|
|
|
- Updated the URL of the Ghostfolio Slack channel |
|
|
|
- Removed the _Ghostfolio in Numbers_ section from the about page |
|
|
|
|
|
|
|
|
|
@ -223,11 +223,17 @@ export class ImportActivitiesService { |
|
|
|
|
|
|
|
for (const key of ImportActivitiesService.DATE_KEYS) { |
|
|
|
if (item[key]) { |
|
|
|
if (isMatch(item[key], 'dd-MM-yyyy')) { |
|
|
|
if (isMatch(item[key], 'dd-MM-yyyy') && item[key].length === '10') { |
|
|
|
// Check length to only match yyyy (and not yy)
|
|
|
|
date = parse(item[key], 'dd-MM-yyyy', new Date()).toISOString(); |
|
|
|
} else if (isMatch(item[key], 'dd/MM/yyyy')) { |
|
|
|
} else if ( |
|
|
|
isMatch(item[key], 'dd/MM/yyyy') && |
|
|
|
item[key].length === '10' |
|
|
|
) { |
|
|
|
// Check length to only match yyyy (and not yy)
|
|
|
|
date = parse(item[key], 'dd/MM/yyyy', new Date()).toISOString(); |
|
|
|
} else if (isMatch(item[key], 'yyyyMMdd')) { |
|
|
|
} else if (isMatch(item[key], 'yyyyMMdd') && item[key].length === '8') { |
|
|
|
// Check length to only match yyyy (and not yy)
|
|
|
|
date = parse(item[key], 'yyyyMMdd', new Date()).toISOString(); |
|
|
|
} else { |
|
|
|
try { |
|
|
|
|
|
@ -0,0 +1,2 @@ |
|
|
|
Date,Code,Currency,Price,Quantity,Action,Fee |
|
|
|
16/09/21,MSFT,USD,298.580,5,buy,19.00 |
1 |
Date |
Code |
Currency |
Price |
Quantity |
Action |
Fee |
2 |
16/09/21 |
MSFT |
USD |
298.580 |
5 |
buy |
19.00 |
|