Skip to content

Add STRDLM and STRESCCHR options to CopyToImport component#2567

Merged
worksofliam merged 1 commit intomasterfrom
fix/ensure_csv_escapes
Mar 17, 2025
Merged

Add STRDLM and STRESCCHR options to CopyToImport component#2567
worksofliam merged 1 commit intomasterfrom
fix/ensure_csv_escapes

Conversation

@worksofliam
Copy link
Copy Markdown
Member

Introduce new options for string delimiter and escape character in the CopyToImport component to ensure lines as escaped and can be parsed correctly.

Fixes #2553.

How to test

  1. Open members that have double quotes in the lines (")
  2. Run the test cases

Signed-off-by: worksofliam <mrliamallan@live.co.uk>
@worksofliam worksofliam temporarily deployed to testing_environment March 7, 2025 18:16 — with GitHub Actions Inactive
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 7, 2025

👋 A new build is available for this PR based on 626dba4.

@worksofliam worksofliam requested a review from a team March 7, 2025 18:18
@RobMKH
Copy link
Copy Markdown

RobMKH commented Mar 17, 2025

Downloaded and tested - it now opens those members that were failing to open.

It's a pass from me.

@worksofliam worksofliam merged commit 69df458 into master Mar 17, 2025
4 checks passed
@sebjulliand sebjulliand deleted the fix/ensure_csv_escapes branch October 23, 2025 20:24
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.

"Some" members will not open

2 participants