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.

Sign up for GitHub

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

Jump to bottom

WIP: Feat/dat 59 admin can unarchive enrollment #1339

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

WIP: Feat/dat 59 admin can unarchive enrollment #1339

wants to merge 4 commits into from

Conversation

Isalafont
Copy link
Contributor

@Isalafont Isalafont commented Nov 30, 2023 •
edited
Loading

Capture d’écran 2023-11-30 à 13 38 06

@Isalafont
Frontend
6e65c94 
- add button and logic to command backend action on event
@Isalafont
logic to click with call to process-event
754b837
@linear Linear
Copy link

linear bot commented Nov 30, 2023

DAT-59 ETQ Admin, je peux désarchiver une habilitation

J'ai rien précisé en front et ça a pas été maquetté : pour moi, best effort tant qu'il n'y a pas de maquette là-dessus, et je vais en parler à eva.spaeter ce jeudi 23.

Quand une habilitation est désarchivée, elle reprend le statut qu'elle avait avant l'archivage.

Pour l'instant, seuls les admins peuvent faire cette action (puisque seuls les admins peuvent voir les habilitations archivées) : à terme, le droit sera ouvert aux instructeurs, mais pas tout de suite.

@Isalafont
create unarchive event
79d55f0 
- add factory trait / adapt test
@Isalafont Isalafont force-pushed the feat/DAT-59-admin_can_unarchive_enrollment branch from ed68256 to 79d55f0 Compare November 30, 2023 12:27
@Isalafont
frontend open confirmationModal and display btn archived for only arc…
bb39229 
…hived enrollments

backend implement logic unarchive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pierreericgarcia pierreericgarcia Awaiting requested review from pierreericgarcia

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Isalafont