Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(api, admin): 3283 - Add manual inscription rights according to dynamic CLE settings #4404

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

Eric013
Copy link
Collaborator

@Eric013 Eric013 commented Oct 2, 2024

Description

Après la date de fermeture des inscriptions, les référent départementaux, référent régionaux, administrateurs CLE, référents de classe n’ont plus la possibilité d’inscrire manuellement des jeunes.

Il existe des exceptions (ex : DROM) pour lesquels les référents doivent avoir la main passé la date de fin d’inscription pour pouvoir continuer à inscrire des jeunes.

Le but était d'ajouter des paramètres dynamiques pour surcharger les droits d'affichage des boutons d'actions liés aux inscriptions, en fonction des différents types de référents, même si la date des inscriptions est dépassée.

Todo

  • Identifier les variables booléennes à créer
  • Rédiger le script de migration
  • Modifier toutes les couches pour inclure ces variables (Schema, DTO, cohortValidator)
  • Modifier la page de paramétrage dynamique (création des composants + modification de l'existant)
  • Modifier la condition d'affichage du bouton dans le Dropdown ActionsList "Inscrire un élève" sur la partie admin

Ticket / Issue

Notion ticket #3283

Testing instructions

  • Rechercher une classe avec un filtre sur une cohorte 2024 avec un statut fermé.
  • Copier l'adresse du référent de classe, la rechercher dans les utilisateurs, puis prendre sa place.
  • Prendre une des classes avec un statut d'inscription fermé.
  • Le dropdown d'action "Inscrire les élèves" est désactivé par défaut avec un tooltip explicatif.
  • Noter le nom de la cohorte (pour la rechercher dans la liste des cohortes sur les paramétrages dynamiques).

Ouvrir une fenêtre de navigation privée

  • Se connecter avec votre compte Modérateur et le sous-rôle "god", puis aller dans les paramétrages dynamiques.
  • Choisir la cohorte sélectionnée dans votre autre fenêtre de navigation.
  • Dans le bloc Inscriptions (phase 0), de nouveaux paramètres sous forme de toggle apparaissent.
  • Activer les paramètres pour les rôles "Référent de classe" et "Administrateur CLE".
  • Sauvegarder.
  • Dans votre autre fenêtre de navigation (celle du référent de classe), rafraîchir la page.
  • Le dropdown d'action "Inscrire les élèves" est maintenant activé et vous pouvez sélectionner l'inscription manuelle.

@Eric013 Eric013 self-assigned this Oct 2, 2024
@achorein achorein added the R&R label Oct 3, 2024
@achorein
Copy link
Collaborator

achorein commented Oct 3, 2024

image

le dropdown "disabled" devrait être gris plutôt que bleu, le DS semblait pourtant le gérer.

@Eric013
Copy link
Collaborator Author

Eric013 commented Oct 7, 2024

image le dropdown "disabled" devrait être gris plutôt que bleu, le DS semblait pourtant le gérer.

@achorein Cela dépend du variant. Pour ce dropdown le variant utilisé est "wired":

{
    native: "h-[38px] py-[9px] text-sm pr-3",
    base: "text-blue-600 border !border-blue-600 hover:bg-blue-50",
    disabled: "text-blue-600/60 !border border-blue-600/60",
    loaderColor: "#2563eb",
 };

Capture d’écran 2024-10-07 à 11 58 35

@Eric013 Eric013 force-pushed the feat-manualInscription-3283 branch from 2596d0b to be09231 Compare October 7, 2024 13:19
@Eric013 Eric013 force-pushed the feat-manualInscription-3283 branch 10 times, most recently from b71fcf6 to a72e0b8 Compare October 24, 2024 14:26
Copy link

codeclimate bot commented Oct 25, 2024

Code Climate has analyzed commit f4fb5d0 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 46.0% (0.0% change).

View more on Code Climate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants