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] Donner la possibilitée aux organisations sans imports de télécharger des attestations (PIX-15612) #10884

Conversation

Alexandre-Monney
Copy link
Contributor

@Alexandre-Monney Alexandre-Monney commented Dec 20, 2024

🎄 Problème

Les EFE n’ont pas d’import, les élèves n’ont donc pas de classe
Nous n’avons pas anticipé ce cas lors de dev de la page attestations, il faut que le prescripteur puisse télécharger toutes les attestations dans les EFE.

🎁 Proposition

Rendre possible le téléchargement des attestations même pour les organisations n'ayant pas d'import

🧦 Remarques

La page sixth-grade.gjs ainsi que son fonctionnement sera surement a revoir, et est prévu pour la v2

🎅 Pour tester

Test de non régression

  • Se connecter avec [email protected]
  • Aller sur l'organisation Attestations
  • Aller sur la page Attestations
  • Constater que l'affichage n'a pas évolué pour ce type d'orga ( Sco avec import )
  • Essayer de télécharger les attestations et constater que le fonctionnement est toujours ISO

Nouveau Test

  • Aller sur l'organisation Sco Classic
  • Vérifier le nouvel affichage pour les organisations sans import
  • Essayer de télécharger l'attestation
  • 🐈‍⬛

@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-15612-make-attestations-page-accessible-to-organization-without-import branch from fff30d3 to cabd955 Compare December 20, 2024 15:36
@Alexandre-Monney Alexandre-Monney added 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally labels Dec 20, 2024
@Alexandre-Monney Alexandre-Monney marked this pull request as ready for review December 20, 2024 15:43
@Alexandre-Monney Alexandre-Monney requested a review from a team as a code owner December 20, 2024 15:43
@Alexandre-Monney Alexandre-Monney force-pushed the pix-15612-make-attestations-page-accessible-to-organization-without-import branch from cabd955 to 94ee8c8 Compare December 20, 2024 15:49
@machestla
Copy link
Contributor

func review ok ✅ 🐕

@machestla machestla added Func Review OK PO validated functionally the PR and removed 👀 Func Review Needed Need PO validation for this functionally labels Dec 20, 2024
@xav-car xav-car closed this Dec 23, 2024
@xav-car xav-car reopened this Dec 23, 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 :

Copy link
Contributor

@xav-car xav-car left a comment

Choose a reason for hiding this comment

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

tech ok

@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-15612-make-attestations-page-accessible-to-organization-without-import branch from 94ee8c8 to 24c5cc6 Compare December 23, 2024 14:38
@pix-service-auto-merge pix-service-auto-merge merged commit ef27ae6 into dev Dec 23, 2024
7 of 8 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-15612-make-attestations-page-accessible-to-organization-without-import branch December 23, 2024 14:44
HEYGUL pushed a commit that referenced this pull request Dec 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants