Skip to content

Inventory Management UI#46

Merged
qiuethan merged 8 commits intomainfrom
feature/inventory-ui
Feb 23, 2026
Merged

Inventory Management UI#46
qiuethan merged 8 commits intomainfrom
feature/inventory-ui

Conversation

@CaellumYHL
Copy link
Contributor

@CaellumYHL CaellumYHL commented Feb 7, 2026

Summary

Added Admin Dashboard with central home and catalogue viewer in the theme of Figma assets. Added buttons for creation/editing of items and developed modals for form submission/confirmation.

Related Issues

Changes

  • Added AdminDashboard, AdminCatalogue, and AddItem/EditItem Modal components (css,tsx)
  • Implemented database formatting to view inventory tables using API endpoints
  • Developed logic for modals and accounted for errors in submission

How to Test

Use/create admin credentials - python app.py createsuperuser
Host the backend and npm run dev
cd into frontend and run npm run test src/pages/admin/AdminCataloguePage.test.tsx

Screenshots / UI (if applicable)

  • Before:
  • After:
image image

Checklist

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

vivjd
vivjd previously requested changes Feb 13, 2026
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.

Overall this looks good, but can you resolve the frontend CI warnings in AdminCatloguePage.tsx and re-request review? Thanks!

Copy link
Contributor Author

@CaellumYHL CaellumYHL left a comment

Choose a reason for hiding this comment

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

Fixed the CI test issues, please let me know if there are any other issues

@qiuethan qiuethan dismissed vivjd’s stale review February 23, 2026 05:17

CI works now

@qiuethan qiuethan force-pushed the feature/inventory-ui branch from 7a815b2 to 6a74648 Compare February 23, 2026 05:33
@qiuethan qiuethan merged commit 3059620 into main Feb 23, 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.

Inventory Management UI

3 participants