Skip to content

fix(feeder): #COCO-2069, import CSV explicit error message#970

Merged
pb-jo merged 1 commit intodevelop-b2schoolfrom
fix-COCO-2069-import-CSV-explicit-error-message
Mar 5, 2026
Merged

fix(feeder): #COCO-2069, import CSV explicit error message#970
pb-jo merged 1 commit intodevelop-b2schoolfrom
fix-COCO-2069-import-CSV-explicit-error-message

Conversation

@pb-jo
Copy link
Contributor

@pb-jo pb-jo commented Mar 5, 2026

Description

Add an explicit error message on CSV import.

Fixes

#COCO-2069

Type of change

Please check options that are relevant.

  • Chore (PATCH)
  • Doc (PATCH)
  • Bug fix (PATCH)
  • New feature (MINOR)

Which packages changed?

Please check the name of the package you changed

  • admin
  • app-registry
  • archive
  • auth
  • cas
  • common
  • communication
  • conversation
  • directory
  • feeder
  • infra
  • portal
  • session
  • test
  • tests
  • timeline
  • workspace

Tests

  1. Describe here the tests you performed
  2. Step by step
  3. With expected results

Reminder

  • Security flaws

  • Performance impacts (think bulk !)

  • Unit tests were replayed

  • Unit tests were added and/or changed

  • I have updated the reminder for the version including my modifications

  • All done ! 😃

@pb-jo pb-jo requested review from jcbe-ode and vbillard91 March 5, 2026 10:08
@pb-jo pb-jo self-assigned this Mar 5, 2026
@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 5, 2026

Quality Gate Failed Quality Gate failed

Failed conditions
D Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@pb-jo pb-jo merged commit a3acfcd into develop-b2school Mar 5, 2026
1 check failed
@pb-jo pb-jo deleted the fix-COCO-2069-import-CSV-explicit-error-message branch March 5, 2026 13:59
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.

3 participants