Skip to content

Rename favorites to saved across full stack#65

Merged
i30101 merged 1 commit into
james/refactor-react-query-hydrationfrom
james/rename-favorites-to-saved
Apr 22, 2026
Merged

Rename favorites to saved across full stack#65
i30101 merged 1 commit into
james/refactor-react-query-hydrationfrom
james/rename-favorites-to-saved

Conversation

@jamesdoh0109
Copy link
Copy Markdown
Collaborator

@jamesdoh0109 jamesdoh0109 commented Apr 3, 2026

Summary

  • Rename all "favorites" terminology to "saved" across backend and frontend
  • Backend: model field (favoritessaved), views (FavoritesSavedListings), serializers (is_favoritedis_saved, favorite_countsaved_count), API endpoints (/favorites//saved/), and tests
  • Frontend: types, server actions (saveListing, unsaveListing, getSavedListings), component variables, and icon (Heart → Bookmark with blue fill)
  • Migration uses RenameField to preserve existing data

Test plan

  • Run backend tests: python manage.py test
  • Run frontend build: npm run build
  • Manually verify save/unsave toggle works on listing detail page
  • Verify Bookmark icon renders correctly (outlined when unsaved, filled blue when saved)
  • Verify /market/saved/ endpoint returns saved listings

🤖 Generated with Claude Code

Replace all "favorites" terminology with "saved" — backend model field,
serializers, views, API endpoints, frontend types, actions, and UI
component. Swap Heart icon for Bookmark. Add migration using RenameField
to preserve existing data.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link
Copy Markdown
Collaborator

@i30101 i30101 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good

@i30101 i30101 merged commit a01ebc6 into james/refactor-react-query-hydration Apr 22, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants