Skip to content

Implemented Box Management API#47

Merged
vivjd merged 5 commits intomainfrom
feature/box-management-api
Feb 13, 2026
Merged

Implemented Box Management API#47
vivjd merged 5 commits intomainfrom
feature/box-management-api

Conversation

@anthonyzhao27
Copy link
Contributor

@anthonyzhao27 anthonyzhao27 commented Feb 7, 2026

Summary

Added Box API support (list/detail), Box description field + migration, and item box assignment via authenticated PATCH; included tests and lint config to keep flake8 clean.

Enables “add to boxes” by letting the system track which items are inside each box and reliably update an item’s box_id, which was a core requirement in the issue.

Related Issues

Changes

  • Create Box entity in the database (ID, Label/Number, Description).
  • Update CollectionItem table to add a nullable box_id foreign key.
  • Implement GET /api/boxes to list all boxes.
  • Implement GET /api/boxes/:id to return box details AND the list of items currently inside.

How to Test

  1. cd backend/inventory && pytest

Checklist

  • Tests added or updated
  • CI passes (lint, tests, build)
  • Documentation updated (if behavior changed)
  • No secrets or credentials committed

@anthonyzhao27 anthonyzhao27 marked this pull request as draft February 7, 2026 22:01
@anthonyzhao27 anthonyzhao27 marked this pull request as ready for review February 8, 2026 01:28
Copy link
Contributor

@vivjd vivjd left a comment

Choose a reason for hiding this comment

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

LGTM! Just as a side note, if you notice issues with the any configs (e.g. lint) in the future, please make a separate PR for it so we can keep our work flow clean!

@vivjd vivjd force-pushed the feature/box-management-api branch from 0094ab9 to f622d63 Compare February 13, 2026 07:45
@vivjd vivjd merged commit b1091db into main Feb 13, 2026
1 check 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.

Box Management API

2 participants