Skip to content

Add support for CSV escape character to EntityDataLoaderImpl#642

Open
puru-khedre wants to merge 2 commits intomoqui:masterfrom
puru-khedre:edl-custom-csv-loader
Open

Add support for CSV escape character to EntityDataLoaderImpl#642
puru-khedre wants to merge 2 commits intomoqui:masterfrom
puru-khedre:edl-custom-csv-loader

Conversation

@puru-khedre
Copy link
Copy Markdown

fixes: #641

Add CSV escape character support to EntityDataLoaderImpl

  • Introduced the csvEscapeChar property to define the escape character used in CSV parsing.
  • Updated CSV parsing configuration to use the defined escape character.

This change allows for handling escape sequences in CSV files, dealing with special characters within quoted fields.

@acetousk
Copy link
Copy Markdown
Member

I think the backwards compatibility of this change is fine, and this is worth merging as is

@puru-khedre
Copy link
Copy Markdown
Author

Hi @acetousk,

Are we waiting for someone else's approval to merge this PR, or is there another issue?

The method we are using for CSV parsing has been deprecated, so I am planning to fix it next.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Issue with CSV Parsing for Embedded Quotes Using EntityDataLoader

2 participants