Skip to content

Yda-6102: refactor csv import in group manager module#387

Merged
lwesterhof merged 7 commits intodevelopmentfrom
YDA-6102-csv-import-parse
Jan 30, 2025
Merged

Yda-6102: refactor csv import in group manager module#387
lwesterhof merged 7 commits intodevelopmentfrom
YDA-6102-csv-import-parse

Conversation

@leonidastri
Copy link
Contributor

@leonidastri leonidastri commented Jan 23, 2025

This is a refactor of csv parsing in Group Manager

  • Replaced manual CSV parsing logic in Group Manager with the PapaParse library.
  • Resolves the issue where commas in column values caused parsing errors.

@leonidastri leonidastri changed the title Yda 6102 csv import parse Yda-6102: refactor csv import in group manager module Jan 23, 2025
@leonidastri leonidastri self-assigned this Jan 23, 2025
@lwesterhof lwesterhof self-requested a review January 28, 2025 07:51
Copy link
Member

@lwesterhof lwesterhof left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@lwesterhof lwesterhof merged commit ad69266 into development Jan 30, 2025
10 checks passed
@lwesterhof lwesterhof deleted the YDA-6102-csv-import-parse branch January 30, 2025 09:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants