feat(conversation): #COCO-4642 add clean messages modal by Romu-C · Pull Request #971 · edificeio/entcore

Romu-C · 2026-03-05T11:03:21Z

Description

Please include a summary of the changes and the related issue.

Fixes

(Enter here Jira or Redmine ticket(s) links)

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

Tests

Describe here the tests you performed
Step by step
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 ! 😃

Copilot

Pull request overview

This PR replaces the existing onboarding modal in the Conversation app with a new "clean messages" modal that informs users about upcoming automatic deletion of old messages (messages older than 2 years not filed in any folder). The modal is designed to display annually after June 1st.

Changes:

Replaced MessageOnboardingModal component (4-screen feature walkthrough) with a new MessageOnboardingCleanModal component (single-screen cleanup announcement with date-based display logic)
Deleted four SVG assets used by the old onboarding modal and added a reference to a new illu-onboarding-clean.svg asset
Added new i18n keys (onboarding.modal.clean.title and onboarding.modal.clean.text) in the French locale file

Reviewed changes

Copilot reviewed 4 out of 9 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
`conversation/frontend/src/routes/root/index.tsx`	Updated import and usage from `MessageOnboardingModal` to `MessageOnboardingCleanModal`
`conversation/frontend/src/routes/root/components/MessageOnboardingCleanModal.tsx`	New modal component with annual display logic (after June 1st) and cleanup messaging
`conversation/frontend/src/routes/root/components/MessageOnboardingModal.tsx`	Deleted old 4-screen onboarding modal
`conversation/frontend/src/assets/illu-onboarding-*.svg` (4 files)	Deleted SVG illustration assets for old onboarding screens
`conversation/backend/src/main/resources/i18n/fr.json`	Added French i18n keys for the new clean messages modal

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-03-05T11:10:47Z

conversation/frontend/src/routes/root/components/MessageOnboardingCleanModal.tsx

+          src: illuOnboardingClean,
+          title: t('onboarding.modal.clean.title'),
+          alt: t('onboarding.modal.clean.title'),
+          text: t('onboarding.modal.clean.text'),


The text property is hardcoded in French directly in the component instead of using the i18n translation function t('onboarding.modal.clean.text'). The i18n key onboarding.modal.clean.text is already defined in fr.json, so this should use t('onboarding.modal.clean.text') for consistency and to support potential future localization.

Suggested change

text: t('onboarding.modal.clean.text'),

text: t('onboarding.modal.clean.text'),

Copilot · 2026-03-05T11:10:47Z

conversation/frontend/src/routes/root/components/MessageOnboardingCleanModal.tsx

+import { DisplayRuleCheckResult } from 'node_modules/@edifice.io/react/dist/modules/modals/OnboardingModal/OnboardingModal';
+import illuOnboardingClean from '~/assets/illu-onboarding-clean.svg';
+import { useI18n } from '~/hooks/useI18n';
+


Importing DisplayRuleCheckResult directly from node_modules/@edifice.io/react/dist/... is fragile and a bad practice. This path references internal build artifacts and could break with any version update of the library. The type should ideally be imported from the library's public API (e.g., @edifice.io/react/modals). If it's not exported from the public API, consider requesting the library maintainers to export it, or define the type locally based on the expected shape instead.

Suggested change

import { DisplayRuleCheckResult } from 'node_modules/@edifice.io/react/dist/modules/modals/OnboardingModal/OnboardingModal';

import illuOnboardingClean from '~/assets/illu-onboarding-clean.svg';

import { useI18n } from '~/hooks/useI18n';

import illuOnboardingClean from '~/assets/illu-onboarding-clean.svg';

import { useI18n } from '~/hooks/useI18n';

type DisplayRuleCheckResult<TCustomState> = {

display: boolean;

nextState: TCustomState;

};

AH ! Peut-être un oubli de ma part => exporter le type correctement depuis le FF si pas déjà fait.

Copilot · 2026-03-05T11:10:47Z

conversation/backend/src/main/resources/i18n/fr.json

  "onboarding.modal.screen4.title": "Les nouveautés à venir",
  "onboarding.modal.title": "La Messagerie fait peau neuve !",
+  "onboarding.modal.clean.title": "Du tri dans vos messages",
+  "onboarding.modal.clean.text": "<strong>L’application Messagerie intégrera bientôt une automatisation de suppression des anciens messages.</strong><br />Afin de limiter l'utilisation d'espace de stockage, les messages de plus de 2 ans seront supprimés s’ils ne sont classés dans aucun dossier.Cette action se fera automatiquement et régulièrement tout au long de l'année. <br />Cela libérera de l'espace sur votre quota de stockage. Cette mesure permettra de libérer jusqu’à 8 tonnes d’équivalent CO2 par an sur toutes nos plateformes. Un bon geste pour la planète ! <br />Cette automatisation sera appliquée à partir du xxx 2026. Pensez à faire du tri dès que possible !",


The text contains "xxx 2026" which appears to be a placeholder for an actual date. This should be replaced with the actual planned date before merging, otherwise users will see "xxx 2026" in the modal.

Suggested change

"onboarding.modal.clean.text": "L’application Messagerie intégrera bientôt une automatisation de suppression des anciens messages. Afin de limiter l'utilisation d'espace de stockage, les messages de plus de 2 ans seront supprimés s’ils ne sont classés dans aucun dossier.Cette action se fera automatiquement et régulièrement tout au long de l'année. Cela libérera de l'espace sur votre quota de stockage. Cette mesure permettra de libérer jusqu’à 8 tonnes d’équivalent CO2 par an sur toutes nos plateformes. Un bon geste pour la planète ! Cette automatisation sera appliquée à partir du xxx 2026. Pensez à faire du tri dès que possible !",

"onboarding.modal.clean.text": "L’application Messagerie intégrera bientôt une automatisation de suppression des anciens messages. Afin de limiter l'utilisation d'espace de stockage, les messages de plus de 2 ans seront supprimés s’ils ne sont classés dans aucun dossier.Cette action se fera automatiquement et régulièrement tout au long de l'année. Cela libérera de l'espace sur votre quota de stockage. Cette mesure permettra de libérer jusqu’à 8 tonnes d’équivalent CO2 par an sur toutes nos plateformes. Un bon geste pour la planète ! Cette automatisation sera appliquée à partir de 2026. Pensez à faire du tri dès que possible !",

Copilot · 2026-03-05T11:10:48Z

conversation/backend/src/main/resources/i18n/fr.json

  "onboarding.modal.screen4.title": "Les nouveautés à venir",
  "onboarding.modal.title": "La Messagerie fait peau neuve !",
+  "onboarding.modal.clean.title": "Du tri dans vos messages",
+  "onboarding.modal.clean.text": "<strong>L’application Messagerie intégrera bientôt une automatisation de suppression des anciens messages.</strong><br />Afin de limiter l'utilisation d'espace de stockage, les messages de plus de 2 ans seront supprimés s’ils ne sont classés dans aucun dossier.Cette action se fera automatiquement et régulièrement tout au long de l'année. <br />Cela libérera de l'espace sur votre quota de stockage. Cette mesure permettra de libérer jusqu’à 8 tonnes d’équivalent CO2 par an sur toutes nos plateformes. Un bon geste pour la planète ! <br />Cette automatisation sera appliquée à partir du xxx 2026. Pensez à faire du tri dès que possible !",


There is a missing space between "dossier." and "Cette" — the text reads dossier.Cette but should be dossier. Cette.

Suggested change

"onboarding.modal.clean.text": "L’application Messagerie intégrera bientôt une automatisation de suppression des anciens messages. Afin de limiter l'utilisation d'espace de stockage, les messages de plus de 2 ans seront supprimés s’ils ne sont classés dans aucun dossier.Cette action se fera automatiquement et régulièrement tout au long de l'année. Cela libérera de l'espace sur votre quota de stockage. Cette mesure permettra de libérer jusqu’à 8 tonnes d’équivalent CO2 par an sur toutes nos plateformes. Un bon geste pour la planète ! Cette automatisation sera appliquée à partir du xxx 2026. Pensez à faire du tri dès que possible !",

"onboarding.modal.clean.text": "L’application Messagerie intégrera bientôt une automatisation de suppression des anciens messages. Afin de limiter l'utilisation d'espace de stockage, les messages de plus de 2 ans seront supprimés s’ils ne sont classés dans aucun dossier. Cette action se fera automatiquement et régulièrement tout au long de l'année. Cela libérera de l'espace sur votre quota de stockage. Cette mesure permettra de libérer jusqu’à 8 tonnes d’équivalent CO2 par an sur toutes nos plateformes. Un bon geste pour la planète ! Cette automatisation sera appliquée à partir du xxx 2026. Pensez à faire du tri dès que possible !",

Copilot · 2026-03-05T11:10:48Z

conversation/frontend/src/routes/root/components/MessageOnboardingCleanModal.tsx

+    return {
+      display: shouldDisplay,
+      nextState: {
+        type: 'Date',
+        value: nowUTC.toISOString(),
+      },


Potential logic issue: nextState is returned unconditionally, even when display is false. If the OnboardingModal component persists nextState regardless of whether the modal is actually shown, then a user who visits before June 1st would have their state saved with the current date. When June 1st arrives, lastDisplayDate.getFullYear() would equal dateToCompare.getFullYear(), so shouldDisplay would remain false and the modal would never be shown to that user. Consider returning nextState only when display is true, or returning undefined for nextState when the modal shouldn't be displayed.

Suggested change

return {

display: shouldDisplay,

nextState: {

type: 'Date',

value: nowUTC.toISOString(),

},

if (shouldDisplay) {

return {

display: true,

nextState: {

type: 'Date',

value: nowUTC.toISOString(),

},

};

}

return {

display: false,

nextState: previousState,

Le nextState est persisté uniquement si toutes les étapes de l'onboarding sont validés.
La proposition de copilot est une sécurité supplémentaire, mais n'apparait pas indispensable.

jcbe-ode

Comme copilot + quelques précisions

jcbe-ode · 2026-03-05T11:17:31Z

conversation/frontend/src/routes/root/components/MessageOnboardingCleanModal.tsx

+          src: illuOnboardingClean,
+          title: t('onboarding.modal.clean.title'),
+          alt: t('onboarding.modal.clean.title'),
+          text: "<strong>L’application Messagerie intégrera bientôt une automatisation de suppression des anciens messages.</strong><br />Afin de limiter l'utilisation d'espace de stockage, les messages de plus de 2 ans seront supprimés s’ils ne sont classés dans aucun dossier.Cette action se fera automatiquement et régulièrement tout au long de l'année. <br />Cela libérera de l'espace sur votre quota de stockage. Cette mesure permettra de libérer jusqu’à 8 tonnes d’équivalent CO2 par an sur toutes nos plateformes. Un bon geste pour la planète ! <br />Cette automatisation sera appliquée à partir du xxx 2026. Pensez à faire du tri dès que possible !",


Utiliser les trads i18n

jcbe-ode · 2026-03-05T11:21:03Z

conversation/frontend/src/routes/root/components/MessageOnboardingCleanModal.tsx

+    return {
+      display: shouldDisplay,
+      nextState: {
+        type: 'Date',
+        value: nowUTC.toISOString(),
+      },


Le nextState est persisté uniquement si toutes les étapes de l'onboarding sont validés.
La proposition de copilot est une sécurité supplémentaire, mais n'apparait pas indispensable.

jcbe-ode · 2026-03-05T11:21:43Z

conversation/frontend/src/routes/root/components/MessageOnboardingCleanModal.tsx

+import { DisplayRuleCheckResult } from 'node_modules/@edifice.io/react/dist/modules/modals/OnboardingModal/OnboardingModal';
+import illuOnboardingClean from '~/assets/illu-onboarding-clean.svg';
+import { useI18n } from '~/hooks/useI18n';
+


AH ! Peut-être un oubli de ma part => exporter le type correctement depuis le FF si pas déjà fait.

sonarqubecloud · 2026-03-24T15:28:13Z

Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

jcbe-ode · 2026-03-25T08:19:52Z

directory/src/main/java/org/entcore/directory/services/impl/DefaultSchoolService.java

+
+				// Deduplicate functions by function name only, ignoring establishment code
+				// Format: "ESTABLISHMENT_CODE$FUNCTION_CODE$FUNCTION_NAME$JOB_CODE$DISCIPLINE"
+				if (functions != null && functions.size() > 0) {


Tout ce code modifié ne fait pas partie de ta PR, je suppose ?

Mince, verifier si le code n'a pas été changé en rebasant

jcbe-ode · 2026-03-25T08:20:36Z

package.json

    "merge2": "^1.0.3",
-    "ode-ngjs-front": "dev",
-    "ode-ts-client": "dev",
+    "ode-ngjs-front": "develop-b2school",


idem, aucune raison de modifier les versions dans une PR de feature

damienromito · 2026-03-26T08:24:02Z

conversation/backend/src/main/resources/i18n/fr.json

  "onboarding.modal.screen4.title": "Les nouveautés à venir",
  "onboarding.modal.title": "La Messagerie fait peau neuve !",
+  "onboarding.modal.clean.title": "Du tri dans vos messages",
+  "onboarding.modal.clean.text": "<strong>L’application Messagerie intégrera bientôt une automatisation de suppression des anciens messages.</strong><br />Afin de limiter l'utilisation d'espace de stockage, les messages de plus de 2 ans seront supprimés s’ils ne sont classés dans aucun dossier.Cette action se fera automatiquement et régulièrement tout au long de l'année. <br />Cela libérera de l'espace sur votre quota de stockage. Cette mesure permettra de libérer jusqu’à 8 tonnes d’équivalent CO2 par an sur toutes nos plateformes. Un bon geste pour la planète ! <br />Cette automatisation sera appliquée à partir du xxx 2026. Pensez à faire du tri dès que possible !",


on peut faire commencer la clé de trad par conversation.onboarding...

damienromito · 2026-03-26T08:26:12Z

directory/src/main/java/org/entcore/directory/services/impl/DefaultSchoolService.java

+
+				// Deduplicate functions by function name only, ignoring establishment code
+				// Format: "ESTABLISHMENT_CODE$FUNCTION_CODE$FUNCTION_NAME$JOB_CODE$DISCIPLINE"
+				if (functions != null && functions.size() > 0) {


Mince, verifier si le code n'a pas été changé en rebasant

Romu-C requested review from Copilot, damienromito and jcbe-ode March 5, 2026 11:03

Copilot started reviewing on behalf of Romu-C March 5, 2026 11:03 View session

Copilot AI reviewed Mar 5, 2026

View reviewed changes

jcbe-ode reviewed Mar 5, 2026

View reviewed changes

vbillard91 force-pushed the develop-b2school branch from c0e5069 to 4524d6c Compare March 9, 2026 09:34

jenkinsEdificePublic force-pushed the develop-b2school branch from ab42bee to 1c68b98 Compare March 9, 2026 13:46

Romu-C force-pushed the feat/COCO-4642_update-onboarding-clean-message branch from 0b983c4 to 865ee69 Compare March 10, 2026 16:26

jcbe-ode force-pushed the develop-b2school branch 2 times, most recently from 3b4c617 to a06ada6 Compare March 16, 2026 15:02

jenkinsEdificePublic force-pushed the develop-b2school branch from 46878dd to 5a2b7a2 Compare March 23, 2026 13:57

damienromito force-pushed the develop-b2school branch from 5a2b7a2 to 73a635f Compare March 23, 2026 15:49

jenkinsEdificePublic force-pushed the develop-b2school branch from 73a635f to 848d378 Compare March 23, 2026 17:28

jenkinsEdificePublic and others added 7 commits March 24, 2026 09:44

chore: prepare next development iteration

2a38f59

fix(directory): #COCO-4881, eliminer doublons fonctions (#924)

b5c6745

chore: update dependencies

e1024ec

chore: update dependencies

df1890f

feat: #COCO-4642 update onboarding clean message

408f552

trasnlation

e76ad80

fix import

50458a5

Romu-C force-pushed the feat/COCO-4642_update-onboarding-clean-message branch from 865ee69 to 50458a5 Compare March 24, 2026 15:25

jcbe-ode requested changes Mar 25, 2026

View reviewed changes

damienromito requested changes Mar 26, 2026

View reviewed changes

	text: t('onboarding.modal.clean.text'),
	text: t('onboarding.modal.clean.text'),

Conversation

Romu-C commented Mar 5, 2026

Description

Fixes

Type of change

Which packages changed?

Tests

Reminder

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Mar 5, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 5, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 5, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 5, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 5, 2026

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

jcbe-ode left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

sonarqubecloud bot commented Mar 24, 2026

Quality Gate passed

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants