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

refactor: global state handling now using zustand. #26

Merged
merged 2 commits into from
Dec 5, 2024

Conversation

ZL-Asica
Copy link
Member

@ZL-Asica ZL-Asica commented Dec 5, 2024

BREAKING CHANGE: Replaced useSessionStorage and context-based state management with zustand.

  • Switched from useSessionStorage to zustand for organization and event state (still session storage).
  • Added real-time listeners using onSnapshot for organization updates, and user auth state changes.
  • Fixed state synchronization for needs and checked items in OrganizationCard.
  • Improved performance and maintainability by centralizing the global state.

BREAKING CHANGE: Replaced useSessionStorage and context-based state management with zustand.

- Switched from useSessionStorage to zustand for organization and event state (still session storage).
- Added real-time listeners using onSnapshot for organization updates, and user auth state changes.
- Fixed state synchronization for needs and checked items in OrganizationCard.
- Improved performance and maintainability by centralizing global state.
Copy link

github-actions bot commented Dec 5, 2024

🚀 Preview for this PR is available at: https://openhands-48a60--pr-26-npwu1ds4.web.app

🧪 Test Results

Tests Passed: All tests passed successfully!


🎨 Prettier Format Check

Prettier: No formatting issues found!

🎨 Lint Check

Lint: No linting issues found!

@ZL-Asica ZL-Asica enabled auto-merge (squash) December 5, 2024 03:00
@ZL-Asica ZL-Asica changed the title fix: global state handling now using zustand. refactor: global state handling now using zustand. Dec 5, 2024
Copy link

github-actions bot commented Dec 5, 2024

🚀 Preview for this PR is available at: https://openhands-48a60--pr-26-npwu1ds4.web.app

🧪 Test Results

Tests Passed: All tests passed successfully!


🎨 Prettier Format Check

Prettier: No formatting issues found!

🎨 Lint Check

Lint: No linting issues found!

@ZL-Asica ZL-Asica merged commit 863dc31 into main Dec 5, 2024
3 checks passed
@ZL-Asica ZL-Asica deleted the fix/global-states branch December 5, 2024 03:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working components context enhancement New feature or request hooks pages size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants