-
Notifications
You must be signed in to change notification settings - Fork 1
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
[Moyens et unités] Finalisation de la fiche unité #867
Conversation
98acc2c
to
3086ea6
Compare
frontend/src/features/ControlUnit/components/ControlUnitListDialog/slice.ts
Outdated
Show resolved
Hide resolved
export enum SeaFrontEnum { | ||
// TODO Why is it not in ALL MAJ? Does the DB has a mix of both cases (`Guadeloupe`, `SOUTH_INDIAN_OCEAN`)? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oui, il y a un mix de majuscule et minuscule en DB
frontend/src/features/MainWindow/useCases/handleModalConfirmation.ts
Outdated
Show resolved
Hide resolved
frontend/src/features/ControlUnit/usesCases/deleteControlUnitResource.ts
Outdated
Show resolved
Hide resolved
frontend/src/features/ControlUnit/usesCases/deleteControlUnit.ts
Outdated
Show resolved
Hide resolved
8c54450
to
7ba50b8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Après discussion avec Claire, je mets une note de review ici : je vais abandonner le système utilisant Redux pour afficher les dialog/modal communes pour utiliser plutôt un système "in-component" çad rendu là où elles sont appelées.
Mais je le ferai sur ma prochaine PR.
bbda84f
to
8402b29
Compare
d8f98cc
to
c394500
Compare
31aebc0
to
e4d06e2
Compare
Related PRs & Issues