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

Use recoil as a store #102

Closed
mjamry opened this issue Dec 27, 2021 · 0 comments · Fixed by #118
Closed

Use recoil as a store #102

mjamry opened this issue Dec 27, 2021 · 0 comments · Fixed by #118
Labels
dependencies Pull requests that update a dependency file frontend high priority refactor code base changes

Comments

@mjamry
Copy link
Owner

mjamry commented Dec 27, 2021 •
edited
Loading

Application should use Recoil as a state manager instead of using React context for dynamic/large data.
For now, it uses context almost everywhere to store state. Which is not good, as even a small state change enforce rerender of most of the components.

https://recoiljs.org/
@mjamry mjamry added frontend refactor code base changes high priority dependencies Pull requests that update a dependency file labels Dec 27, 2021
@mjamry mjamry added this to To do in plan-your-trip via automation Dec 27, 2021
@mjamry mjamry removed this from To do in plan-your-trip Jan 19, 2022
mjamry added a commit that referenced this issue Feb 16, 2022
@mjamry
#102 Added recoil package
26d584d
mjamry added a commit that referenced this issue Feb 16, 2022
@mjamry
#102 Some corrections to the logger service to better display errors
d23663e
mjamry added a commit that referenced this issue Feb 16, 2022
@mjamry
#102 Replaced app state
ce17a11
mjamry added a commit that referenced this issue Feb 16, 2022
@mjamry
#102 Replaced user state
5a4fba6
mjamry added a commit that referenced this issue Feb 17, 2022
@mjamry
#102 Updated locations state provider
527fe42
mjamry added a commit that referenced this issue Feb 17, 2022
@mjamry
#102 Updated modal state
dd5990e
mjamry added a commit that referenced this issue Feb 17, 2022
@mjamry
#102 Updated notifications state
c22a15a
mjamry added a commit that referenced this issue Feb 18, 2022
@mjamry
#102 Updated share state
3f9453c
mjamry added a commit that referenced this issue Feb 18, 2022
@mjamry
#102 Updated location form state
2d2bf51
mjamry added a commit that referenced this issue Feb 18, 2022
@mjamry
#102 Updated plans state
dc2fa09
mjamry added a commit that referenced this issue Feb 18, 2022
@mjamry
#102 Added diagnostics to recoil state changes
31fefd6
@mjamry mjamry mentioned this issue Feb 18, 2022
Merged
mjamry added a commit that referenced this issue Feb 18, 2022
@mjamry
Merge pull request #118 from mjamry/change/#102-use-recoil
81aa875 
Change/#102 use recoil
@mjamry mjamry closed this as completed Feb 25, 2022
@mjamry mjamry mentioned this issue Mar 31, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
dependencies Pull requests that update a dependency file frontend high priority refactor code base changes
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Change/#102 use recoil mjamry/plan-your-trip
1 participant
@mjamry