feat(lists): add user lists feature with ownership validation - #188
Merged
Conversation
Develop to Main
Develop to Main
Develop to Main
Develop to Main
Develop to Main
Develop to Main
Develop to Main
Develop to Main
Develop to Main
Develop to Main
Develop to Main
Develop to Main
Develop to Main
Develop to Main
Develop to Main
Develop to Main
Develop to Main
Develop to Main
Develop to Main
Develop to Main
Develop to Main
Develop to Main
Develop to Main
Develop to Main
Develop to Main
Develop to Main
Develop to Main
Develop to Main
Develop to Main
Develop to Main
Develop to Main
Develop to Main
Develop to Main
Develop to Main
- Move /dashboard/lists to /lists (protected) - Move /dashboard/lists/[id]/edit to /lists/[id]/edit - Add /lists/new page following /spots/new pattern - Remove old /dashboard/lists directory - Remove unused (public)/lists page
- Dashboard: link to /lists/new instead of /dashboard/lists - CreateListModal: redirect to /lists/:id/edit after creation - ListDetail: edit link points to /lists/:id/edit
- Strapi returns id as number; AuthContext was storing it raw - User type declares id: string, fix both signIn and loadUserData paths - Fixes === comparisons with string values like URL params
- User.id from AuthContext vs userId from URL params can be different types - Use String() on both sides for reliable comparison
- Add createListItem service and hook - Update list types with owner avatar support - Improve ListDetail with owner link and delete from detail - Refactor ListItemForm, ListItemList for better UX - Add useUpdateList, useDeleteListItem hooks - Add loading states to useCreateList, useDeleteList - Add staleTime to list queries
- Add UserBasics type with proper fields - Update User type - Update getUser service - Fix Edit component usage
Owner
Author
|
This pull request has been deployed to Vercel.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds user-curated lists (wish/like/want/recommend) with public discovery, dashboard management, and profile integration.
Changes
New Pages
/lists— public discovery with type filter pills/lists/[id]— public detail view with items/dashboard/lists— dashboard management with edit/delete/dashboard/lists/[id]/edit— full item CRUD with image uploadNew Feature Module
src/features/lists/— types, 9 services, 9 TanStack Query hooks, 6 components, barrel exportModified Files
Security Fix