Skip to content

feat(admin): #IMPULS-5956 ajoute l'option de duplication du mode d'authentification par défaut - #1064

Merged
damienromito merged 1 commit into
develop-b2schoolfrom
feat-IMPULS-5956-duplicate-auth-mode
Jun 30, 2026
Merged

feat(admin): #IMPULS-5956 ajoute l'option de duplication du mode d'authentification par défaut#1064
damienromito merged 1 commit into
develop-b2schoolfrom
feat-IMPULS-5956-duplicate-auth-mode

Conversation

@damienromito

@damienromito damienromito commented Jun 30, 2026

Copy link
Copy Markdown
Member

Description

Ajoute une option de duplication du mode d'authentification par défaut des comptes en alimentation automatique dans le formulaire de duplication de paramétrage d'une structure (ADMC).

Lorsque cette option est cochée, le paramètre setDefaultAuth est transmis à l'API /directory/structure/:id/duplicate, qui applique la configuration defaultAuthModes de la structure source aux structures cibles.

Fixes

IMPULS-5956 (sous-tâche front de IMPULS-5853)

La partie back (setDefaultAuth côté API) a été livrée dans #1054 (IMPULS-5955).

Type of change

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

Which packages changed?

  • admin

Tests

  1. Se connecter en tant qu'ADMC
  2. Aller sur une structure dont le mode d'authentification par défaut a été configuré (ex. mode FEDERATED pour les enseignants)
  3. Aller dans l'onglet Informations → section Dupliquer le paramétrage
  4. Cocher "Mode d'authentification par défaut des comptes en alimentation automatique" et saisir un UAI cible valide
  5. Valider → vérifier que la structure cible reçoit bien les mêmes defaultAuthModes que la source

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 ! 😃

…thentification par défaut

Permet de dupliquer la configuration du mode d'authentification par
défaut des comptes en alimentation automatique lors de la duplication
de paramétrage d'une structure.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
C Reliability 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

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Ajoute côté admin une nouvelle option dans le formulaire ADMC de duplication du paramétrage d’une structure, permettant de dupliquer également le mode d’authentification par défaut (comptes en alimentation automatique) vers les structures cibles via l’option API setDefaultAuth.

Changes:

  • Ajout d’un booléen defaultAuth dans les paramètres de duplication et prise en compte dans la validation du formulaire.
  • Envoi du flag setDefaultAuth dans le payload de duplication (/directory/structure/:id/duplicate).
  • Ajout du libellé i18n FR pour la nouvelle option.

Reviewed changes

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

File Description
admin/src/main/ts/src/app/management/structure-informations/structure-informations.service.ts Ajoute setDefaultAuth dans les options envoyées à l’API de duplication.
admin/src/main/ts/src/app/management/structure-informations/structure-informations.component.ts Ajoute defaultAuth aux réglages et l’intègre à canDuplicateSettings().
admin/src/main/ts/src/app/management/structure-informations/structure-informations.component.html Ajoute la case à cocher “default auth” au formulaire de duplication.
admin/src/main/resources/i18n/fr.json Ajoute la traduction FR du libellé de la nouvelle option.

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

name="defaultAuth"
[(ngModel)]="settings.defaultAuth"
/>
</ode-form-field>

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

En reponse à sonarcube

ode-form-field génère un sans attribut for et n'expose pas de binding pour en ajouter un — la liaison sémantique for/id n'est donc pas réalisable sans modifier le composant de la librairie. Si l'accessibilité est l'objectif, il faudrait soit faire évoluer ode-form-field pour accepter un [inputId], soit remplacer ce champ par un + input natif.

@damienromito
damienromito requested a review from jcbe-ode June 30, 2026 15:45
@damienromito
damienromito merged commit a23117c into develop-b2school Jun 30, 2026
1 of 2 checks passed
@damienromito
damienromito deleted the feat-IMPULS-5956-duplicate-auth-mode branch June 30, 2026 16:01
jenkinsEdificePublic pushed a commit that referenced this pull request Jul 3, 2026
…thentification par défaut (#1064)

Permet de dupliquer la configuration du mode d'authentification par
défaut des comptes en alimentation automatique lors de la duplication
de paramétrage d'une structure.

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
jcbe-ode pushed a commit that referenced this pull request Jul 3, 2026
…thentification par défaut (#1064)

Permet de dupliquer la configuration du mode d'authentification par
défaut des comptes en alimentation automatique lors de la duplication
de paramétrage d'une structure.

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
vbillard91 pushed a commit that referenced this pull request Jul 6, 2026
…thentification par défaut (#1064)

Permet de dupliquer la configuration du mode d'authentification par
défaut des comptes en alimentation automatique lors de la duplication
de paramétrage d'une structure.

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
jenkinsEdificePublic pushed a commit that referenced this pull request Jul 7, 2026
…thentification par défaut (#1064)

Permet de dupliquer la configuration du mode d'authentification par
défaut des comptes en alimentation automatique lors de la duplication
de paramétrage d'une structure.

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.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.

3 participants