Skip to content

fix: remove trailing separators from CSV export - #8363

Open
Kumar-s75 wants to merge 1 commit into
nextcloud:mainfrom
Kumar-s75:fix/csv-export-trailing-separators
Open

fix: remove trailing separators from CSV export#8363
Kumar-s75 wants to merge 1 commit into
nextcloud:mainfrom
Kumar-s75:fix/csv-export-trailing-separators

Conversation

@Kumar-s75

Copy link
Copy Markdown

Summary

  • format CSV list values with join(', ') instead of trimming a manually appended separator
  • remove trailing commas from tag and assigned-user fields
  • preserve CSV escaping for embedded double quotes
  • add tests for multiple values, quoted values, and empty lists

Verification

  • npx jest src/services/BoardApi.spec.js --runInBand --config='{"rootDir":".","moduleFileExtensions":["js"],"transform":{"^.+\\.js$":"babel-jest"}}'
  • npx eslint src/services/BoardApi.js src/services/BoardApi.spec.js
  • npm run build

All checks passed. The build reports existing Sass deprecation and bundle-size warnings.

Closes #8361

Signed-off-by: Kumar Saurabh <kumarsaurabh27d@gmail.com>
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.

CSV export leaves trailing commas in tags and assigned-user fields

1 participant