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

[Tech] Normalise l'architecture Redux & RTK #917

Merged
merged 1 commit into from
Oct 30, 2023
Merged

[Tech] Normalise l'architecture Redux & RTK #917

merged 1 commit into from
Oct 30, 2023

Conversation

ivangabriele
Copy link
Member

Related Pull Requests & Issues

None

  • Tests E2E (Cypress)

@ivangabriele ivangabriele marked this pull request as draft October 25, 2023 10:52
@ivangabriele ivangabriele force-pushed the ivan/normalize-redux-and-rtk-architecture branch 3 times, most recently from 7082af1 to 78e7b12 Compare October 25, 2023 14:53
@ivangabriele ivangabriele force-pushed the ivan/normalize-redux-and-rtk-architecture branch from 78e7b12 to d4980e3 Compare October 25, 2023 14:54
@ivangabriele ivangabriele marked this pull request as ready for review October 25, 2023 15:43
claire2212
claire2212 approved these changes Oct 26, 2023
View reviewed changes
Copy link
Collaborator

@claire2212 claire2212 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Est-ce tu as bien vérifié que tout se mettait bien à jour (gestion du cache) ? Genre quand tu modifies une mission, le tableau et la carte se mettent bien à jour? idem pour les reportings

@ivangabriele
Copy link
Member Author

Est-ce tu as bien vérifié que tout se mettait bien à jour (gestion du cache) ? Genre quand tu modifies une mission, le tableau et la carte se mettent bien à jour? idem pour les reportings

Je viens de tester et ça se met bien à jour ^^.

@ivangabriele ivangabriele merged commit 05ffc08 into main Oct 30, 2023
9 checks passed
@ivangabriele ivangabriele deleted the ivan/normalize-redux-and-rtk-architecture branch October 30, 2023 07:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@claire2212 claire2212 claire2212 approved these changes

@thoomasbro thoomasbro Awaiting requested review from thoomasbro

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

Successfully merging this pull request may close these issues.
2 participants
@ivangabriele @claire2212