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: add crud event #46

Merged
merged 7 commits into from
Jan 23, 2024
Merged

Feat: add crud event #46

merged 7 commits into from
Jan 23, 2024

Conversation

MaGOs92
Copy link
Collaborator

@MaGOs92 MaGOs92 commented Nov 24, 2023

Gestion des évènements dans BAL-admin

@MaGOs92 MaGOs92 requested a review from fufeck November 24, 2023 16:07
@MaGOs92 MaGOs92 marked this pull request as ready for review November 28, 2023 15:33
Copy link
Collaborator

@fufeck fufeck left a comment

Choose a reason for hiding this comment

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

J'aurais bien séparé l'update de mes-adresses-api v2 (avec les services openapi), de la mise a jour du lint/prettier et de l'ajout du crud event. La PR est un peu fourre tout

Sinon pour l'ajout du crud event, Comme on avait parlé, la partie serveur express va disparaitre (car pas besoin de protéger les token avec keycloack) donc je l'aurais directement intégré coté nextJS

@fufeck fufeck self-requested a review December 13, 2023 17:26
Copy link
Collaborator

@fufeck fufeck left a comment

Choose a reason for hiding this comment

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

Capture d’écran 2023-12-13 à 18 19 15

J'ai créer un event pour tester, cela a bloqué adresse.gouv. Après que la popup de création se soit afficher je n'ai pas été redirigé sur l'acueille des events et j'ai donc pu continuer a appuyer sur sauvegarder ce qui n'a étrangement pas créer plusieurs events?

Sinon il faudrait juste trier les events sur la page d'acueille, celui que j'ai créé c'est retrouvé toute a la fin

Capture d’écran 2023-12-13 à 18 29 22

@MaGOs92
Copy link
Collaborator Author

MaGOs92 commented Jan 4, 2024

Capture d’écran 2023-12-13 à 18 19 15 J'ai créer un event pour tester, cela a bloqué adresse.gouv. Après que la popup de création se soit afficher je n'ai pas été redirigé sur l'acueille des events et j'ai donc pu continuer a appuyer sur sauvegarder ce qui n'a étrangement pas créer plusieurs events?

Sinon il faudrait juste trier les events sur la page d'acueille, celui que j'ai créé c'est retrouvé toute a la fin

Capture d’écran 2023-12-13 à 18 29 22

Bien vu pour le bug sur le site adresse! En faite il y'avait bien un redirect mais sur la page de modification de l'évènement, donc en cliquant plusieurs fois tu modifiais le même event. Je l'ai remplacé par un redirect vers la liste des events c'est plus pratique.
J'ai aussi ajouté le tri par date d'évènement : Du plus proche au plus éloigné

@MaGOs92 MaGOs92 force-pushed the gfay_feat_add-crud-event branch from fe6f664 to aa3ad65 Compare January 4, 2024 14:35
@MaGOs92 MaGOs92 force-pushed the gfay_feat_add-crud-event branch from aa3ad65 to b672dc1 Compare January 4, 2024 14:38
@MaGOs92 MaGOs92 requested a review from fufeck January 9, 2024 15:50
Copy link
Collaborator

@fufeck fufeck left a comment

Choose a reason for hiding this comment

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

C'est good ;)

components/events/event-form.tsx Outdated Show resolved Hide resolved
@fufeck fufeck self-requested a review January 17, 2024 14:18
@MaGOs92 MaGOs92 merged commit 72dfa80 into master Jan 23, 2024
1 check passed
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.

2 participants