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

[Mission] Résolution du bug de sauvegarde de mission durant la navigation avec les signalements #874

Merged
merged 2 commits into from
Oct 12, 2023

Conversation

claire2212
Copy link
Collaborator

j'ai fait un rapide bugfix mais avec l'diée de repasser le multi missions comme les signalements (donc ne plus avoir de missionState) plus tard

@claire2212 claire2212 added the bug Something isn't working label Oct 12, 2023
@claire2212 claire2212 self-assigned this Oct 12, 2023
@claire2212 claire2212 changed the base branch from main to claire/fix-empty-themes-in-mission October 12, 2023 09:32
@@ -28,7 +29,8 @@ export function SearchReportings() {
dispatch(addReporting(ReportingContext.MAP))
}

const toggleReportingsWindow = () => {
const toggleReportingsWindow = async () => {
await dispatch(saveMissionInLocalStore())
Copy link
Member

Choose a reason for hiding this comment

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

Juste un micro-détail mais si j'ai bien compris, celle ligne fonctionne parce qu'il n'y que missions et reportings. S'il y avait d'autres rubriques, il faudrait soit le faire avant que ce toggle ne soit appelé (plus propre), soit vérifier dans ce toggle que ce soit bien une mission que l'on est train de "quitter", non ?

Copy link
Collaborator Author

@claire2212 claire2212 Oct 12, 2023

Choose a reason for hiding this comment

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

ouais dans l'idée c'est ça. Je voulais au début sauvegarder la mission grâce à la navigation de la side window mais comme c'est pas une vraie navigation c'est mort.
Mais comme j'expliquais au-dessus c'est un peu un fix temporaire en attendant de gérer les mission comme les signalement: plus de missionState et seulement un store avec toutes les missions dont une active

Base automatically changed from claire/fix-empty-themes-in-mission to main October 12, 2023 13:58
@claire2212 claire2212 merged commit 844343a into main Oct 12, 2023
9 checks passed
@claire2212 claire2212 deleted the claire/fix-save-mission branch October 12, 2023 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
2 participants