-
Notifications
You must be signed in to change notification settings - Fork 0
feat: added item status enum, outfit location logging #169
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
Conversation
|
Pull Request Smart Comments
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR introduces an item status system to replace the binary archived state, and adds location tracking for outfits. The item status enum supports three states: 'available', 'withheld', and 'retired', providing more granular control over item availability. Additionally, outfits can now record location coordinates when created.
Key Changes:
- Replaced binary
isArchivedfield with a three-statestatusenum for items - Added geolocation capture and storage for outfit creation
- Removed archive-specific filtering logic in favor of status-based filtering
Reviewed Changes
Copilot reviewed 16 out of 19 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| next/src/lib/types.ts | Defines ItemStatus type and constants for the new status system |
| next/src/lib/hooks/useLocation.ts | New hook for requesting and managing geolocation data |
| next/src/lib/hooks/useItemSearch.ts | Removed archiveFilter parameter and simplified filtering logic |
| next/src/lib/client.ts | Updated API client to use updateItemStatus instead of archiveItem |
| next/src/components/OutfitList.tsx | Added location pin icon display for outfits with coordinates |
| next/src/components/Item.tsx | Updated to display status labels instead of archived state |
| next/src/components/AddOutfitModal.tsx | Integrated location capture when creating outfits |
| next/src/app/(authenticated)/items/page.tsx | Updated sorting and context menu to support three item statuses |
| hono/test/utils/factory/items.ts | Added status field to item test factories |
| hono/src/services/outfits.ts | Updated outfit service to handle location fields and status-based filtering |
| hono/src/services/items.ts | Replaced archive endpoint with status update via PUT, updated sorting |
| hono/src/schema.ts | Added itemStatus enum and location fields to database schema |
| hono/src/drizzle/0007_concerned_nitro.sql | Migration script for status enum and location columns |
Files not reviewed (2)
- hono/package-lock.json: Language not supported
- next/package-lock.json: Language not supported
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…calplatforms/shafa into radison/item-status-outfit-location
No description provided.