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

feat: fetch organizations from firestore + update saved to firestore #13

Merged
merged 2 commits into from
Nov 21, 2024

Conversation

marycaserio
Copy link
Contributor

Please include a summary of the changes and the related issue.

Type of change

  • Bug fix
  • New feature
  • Code refactor
  • Breaking change
  • Style update
  • Documentation update

Checklist

  • I have performed a self-review of my code
  • Single file within 200 lines of codes
  • Comments has added
  • My changes generate no new warnings

Copy link

🚀 Preview for this PR is available at: https://openhands-48a60--pr-13-h0eziwf4.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!

- Remove extra `SavedContext` and corresponde `useSaved` hook.
- Create `useSavedOrgs` hook to directly use `UserContext` to handle global state.

- Add auto-fetch `organizationProfiles` into `UserContext` every time session refresh and listen to firebase updates.
- Directly saved to `Session storage` which will save quotas and ensure speed.
Copy link

🚀 Preview for this PR is available at: https://openhands-48a60--pr-13-h0eziwf4.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 afec091 into main Nov 21, 2024
3 checks passed
@ZL-Asica ZL-Asica deleted the firebase branch November 21, 2024 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants