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

Refactor useTheme to use useManagedStorageState #9227

Open

mnholtz opened this issue Oct 3, 2024 · 0 comments · May be fixed by #9228

Assignees

Labels

developer experience

Collaborator

mnholtz commented Oct 3, 2024

Follow-up of #9224

Re @twschiller's comment:

NIT: instead of using useAsyncState with readManagedStorageByKey, we might consider using useManagedStorageState (and standardizing that hook return value use theAsyncState shape)

mnholtz added the developer experience label

mnholtz mentioned this issue

#9222 Add policy for hiding the sidebar logo #9224

Merged

mnholtz self-assigned this

mnholtz linked a pull request

that will close this issue

#9227 Refactor useTheme to use useManagedStorageState #9228

Open

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment