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

[FEATURE] Permettre aux prescripteurs de télécharger les attestations de leurs élèves (PIX-13827) #10531

Merged

Conversation

Alexandre-Monney
Copy link
Contributor

@Alexandre-Monney Alexandre-Monney commented Nov 12, 2024

🍂 Problème

Dans le cadre de l'Epix sur les attestations, nous devons mettre a dispo une page dans PixOrga pour que le prescripteur puisse télécharger les attestations de ses élèves ( Dans le cadre des attestations de 6eme )

🌰 Proposition

Ajouter une page /attestations dans PixOrga. Cette page utilisera un multiselect afin de pouvoir choisir plusieurs classes, et télécharger les attestations dispo de ces classes.
L'affichage de cette page est conditionnée par l'activation ou non de la feature "ATTESTATION_MANAGEMENT" qui est créée et ajoutée au cours de cette PR.

🎃 Remarques

Dans un premier temps, le composant est spécifique au cas des attestations de 6eme. On veut éviter de penser trop loin tout en ne sachant pas comment seront adresser les prochaines. Un refacto, ou des reflexions viendront pour la prochaine attestation ( voir la suivante ... )

🪵 Pour tester

-> Se connecter avec orga-admin@example.net.

-> Aller sur l'orga SCO SIECLE ( feature activée )

  • Vérifier la présence de l'entrée "Attestations"
  • Aller sur la page, et essayer de télécharger les attestations ( Seul la classe de 6eme en a dans les seeds )

-> Aller sur une autre orga sans la feature

  • Vérifier que l'entrée n'est plus présente
  • Essayer d'aller sur /attestations
  • Vérifier que ça n'affiche pas la page

🐈‍⬛

@Alexandre-Monney Alexandre-Monney self-assigned this Nov 12, 2024
@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@Alexandre-Monney Alexandre-Monney force-pushed the pix-13827-get-attestations-list-for-learners branch from 6b6ad87 to 9d04672 Compare November 13, 2024 09:30
@Alexandre-Monney Alexandre-Monney added team-evaluation PR relatives à l'expérience d'évaluation team-prescription labels Nov 13, 2024
@Alexandre-Monney Alexandre-Monney force-pushed the pix-13827-get-attestations-list-for-learners branch from 9d04672 to e7a89db Compare November 13, 2024 10:33
@Alexandre-Monney Alexandre-Monney marked this pull request as ready for review November 13, 2024 10:33
Copy link
Contributor

@La-toile-cosmique La-toile-cosmique left a comment

Choose a reason for hiding this comment

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

:+1

@Faraopix
Copy link
Contributor

Est-ce qu'on peut "juste" retirer cette phrase "Vous pouvez également télécharger toutes les attestations de votre établissements." car on ne propose pas de tout sélectionner encore ?
Func ok sinon

@Faraopix Faraopix added Func Review OK PO validated functionally the PR and removed 👀 Func Review Needed labels Nov 14, 2024
@Alexandre-Monney Alexandre-Monney force-pushed the pix-13827-get-attestations-list-for-learners branch from 52769d3 to c6a90ef Compare November 14, 2024 16:38
@Faraopix Faraopix added 👀 Func Review Needed and removed Func Review OK PO validated functionally the PR labels Nov 14, 2024
@Faraopix
Copy link
Contributor

Faraopix commented Nov 14, 2024

Après échange avec @Alexandre-Monney et @pierrepougetpix :
peut-on placer le bouton à droite pour ne pas cacher le bouton lorsqu'on sélectionne les classes ?
& garder la largeur du select fixe ?

@Alexandre-Monney Alexandre-Monney force-pushed the pix-13827-get-attestations-list-for-learners branch 3 times, most recently from cf1c3b0 to aefc95b Compare November 15, 2024 10:29
@Faraopix Faraopix added Func Review OK PO validated functionally the PR 🚀 Ready to Merge and removed 👀 Func Review Needed labels Nov 15, 2024
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-13827-get-attestations-list-for-learners branch from aefc95b to ec591a7 Compare November 15, 2024 10:41
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-13827-get-attestations-list-for-learners branch from ec591a7 to 1533294 Compare November 15, 2024 10:47
@pix-service-auto-merge pix-service-auto-merge merged commit 2af419c into dev Nov 15, 2024
8 of 9 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-13827-get-attestations-list-for-learners branch November 15, 2024 10:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Func Review OK PO validated functionally the PR 🚀 Ready to Merge team-evaluation PR relatives à l'expérience d'évaluation team-prescription Tech Review OK
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants