-
Notifications
You must be signed in to change notification settings - Fork 5
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
User profile and settings #16
Comments
See #79 (review) for some issues (item 1, 2, and 3 in that list) that should be addressed in this feature that were left out of the initial #13 feature completion. The initial load of user settings should ideally not rely on a client side call to Either way, re-hydration based on user settings or changes to the user settings should be supported, so it might make sense to progressively improve rather than jumping to full SSR unless warranted. If needed we could split this out into two or more issues (create and define user settings and page, allow page to be dynamic when adjusting user preferences, support SSR for logged in user's user preferences) to be addressed individually if this proves to be too much for a single issue. |
Depends on sublinks/sublinks-api#398 |
After user auth flow(issue #13) has been added to the UI we can expand on the user features and create the user profile and settings pages.
The text was updated successfully, but these errors were encountered: