Skip to content

feat: add "/inventory-catalog" endpoint#91

Merged
WChunter21 merged 6 commits intomainfrom
cc-101/inventory-catalog
Feb 18, 2026
Merged

feat: add "/inventory-catalog" endpoint#91
WChunter21 merged 6 commits intomainfrom
cc-101/inventory-catalog

Conversation

@kevbang
Copy link
Copy Markdown
Collaborator

@kevbang kevbang commented Feb 12, 2026

CC-101

  • Adding an "Item/Inventory" catalog so a restaurant owner can see the possible items that can be inside of a store. This does not keep track of the actual stock value of each item or ingredient:
image

For details, refer to Copilot review

### CC-101

- Adding an "Item/Inventory" catalog so a restaurant owner can see the possible items that can be inside of a store. This does not keep track of the actual stock value of each item or ingredient
@kevbang kevbang marked this pull request as draft February 12, 2026 18:01
@WChunter21
Copy link
Copy Markdown
Collaborator

What's the Validation error about?

@kevbang kevbang marked this pull request as ready for review February 18, 2026 16:42
@kevbang kevbang requested a review from Copilot February 18, 2026 16:42
Copy link
Copy Markdown

Copilot AI left a 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 adds a new inventory catalog feature to help restaurant owners manage their ingredient catalog. The feature introduces a new Blazor component at /inventory-catalog that provides a UI for viewing, searching, filtering, sorting, and managing catalog items. The catalog tracks item metadata (name, SKU, category, unit, price, supplier, notes) but does not track actual stock quantities—that functionality is separate.

Changes:

  • Added InventoryCatalogDTO with unit tests for managing catalog item data
  • Created a full-featured Blazor UI component with search, filtering, sorting, pagination, and CRUD operations
  • Added navigation menu item for the new inventory catalog page
  • Removed hardcoded credentials from appsettings.json (security improvement)
  • Removed Playwright test project from solution and CI workflow

Reviewed changes

Copilot reviewed 8 out of 9 changed files in this pull request and generated 19 comments.

Show a summary per file
File Description
CulinaryCommandApp/Inventory/DTOs/InventoryCatalogDTO.cs New DTO defining the structure for inventory catalog items with properties for ID, name, SKU, category, unit, price, supplier, and notes
CulinaryCommandUnitTests/Inventory/DTOs/InventoryCatalogDTOTests.cs Comprehensive unit tests for the InventoryCatalogDTO covering default values, property setters, null handling, and decimal precision
CulinaryCommandApp/Inventory/Pages/Inventory/InventoryCatalog.razor Interactive Blazor component implementing the catalog UI with search, filtering, sorting, pagination, and modal-based CRUD operations
CulinaryCommandApp/Inventory/Pages/Inventory/InventoryCatalog.razor.css Comprehensive styling for the catalog page with responsive design, custom color scheme, and modal styling
CulinaryCommandApp/Components/Layout/NavMenu.razor Added navigation link to the new Inventory Catalog page
CulinaryCommandApp/appsettings.json Removed hardcoded environment variables including a Cognito client secret (positive security change)
CulinaryCommandApp/CulinaryCommand.sln Removed Playwright test project reference from the solution
.github/workflows/validate.yml Commented out Playwright setup and installation steps in the CI workflow

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread CulinaryCommandApp/Inventory/DTOs/InventoryCatalogDTO.cs
Comment thread CulinaryCommandApp/Inventory/Pages/Inventory/InventoryCatalog.razor
Comment thread CulinaryCommandApp/Inventory/Pages/Inventory/InventoryCatalog.razor
Comment thread CulinaryCommandApp/Inventory/DTOs/InventoryCatalogDTO.cs
Comment thread CulinaryCommandApp/Inventory/Pages/Inventory/InventoryCatalog.razor
Comment thread CulinaryCommandApp/Inventory/Pages/Inventory/InventoryCatalog.razor
Comment thread CulinaryCommandApp/Inventory/DTOs/InventoryCatalogDTO.cs
Comment thread CulinaryCommandApp/Inventory/Pages/Inventory/InventoryCatalog.razor
Comment thread CulinaryCommandApp/Inventory/Pages/Inventory/InventoryCatalog.razor
Comment thread CulinaryCommandApp/Inventory/Pages/Inventory/InventoryCatalog.razor
@WChunter21 WChunter21 merged commit b7777c2 into main Feb 18, 2026
7 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.

3 participants