Skip to content

ShravyaKudlu-KitchenOrders-4804#5363

Open
ShravyaKudlu wants to merge 7 commits into
developmentfrom
shravya/bugfix/kitchenorders
Open

ShravyaKudlu-KitchenOrders-4804#5363
ShravyaKudlu wants to merge 7 commits into
developmentfrom
shravya/bugfix/kitchenorders

Conversation

@ShravyaKudlu

@ShravyaKudlu ShravyaKudlu commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Description

Bugfixes and feature additions as outlined in the requirements.
This is a bugfix realted to PR #4804
swappy-20260703-000224

Main changes explained:

  • Fixed dark mode styling issues
  • Updated order status flow:
    • Ordered → action: Mark as Received
    • Received → action: Mark as Stocked
    • Stocked → no action; orders can still be navigated/viewed
  • Added ability to place a new order
  • Added suppliers list (using mock data)
  • Added order/item details views
image

How to test:

  1. check into current branch
  2. do npm install and ... to run this PR locally
  3. Clear site data/cache
  4. log as admin user
  5. go to http://localhost:5173/kitchenandinventory/orders
  6. verify function the changes
  7. verify this new feature works in dark mode

Screenshots or videos of changes:

Note:

2 things are out of scope
The "autogenerate inventory list" button is present in the UI but not functional — it isn't connected to the backend yet.
The screenshot of the UI shows urgent, but I have mentioned it as note, again, as its not connected to the backend

kanishkagarwal6101 and others added 7 commits November 19, 2025 16:53
- Create OrdersPage component with 4 summary cards
- Implement PurchaseOrders section with search functionality
- Add order status tags (Ordered, Received, Stocked)
- Add action buttons (Mark as Received, Mark as Stocked)
- Implement dark mode support using CSS modules
- Add route /kitchenandinventory/orders

Implements Deliverable 2 - Phase 6 Kitchen Inventory Management
@netlify

netlify Bot commented Jul 3, 2026

Copy link
Copy Markdown

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 57be3c5
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/6a4746827218a80008b81426
😎 Deploy Preview https://deploy-preview-5363--highestgoodnetwork-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@sonarqubecloud

sonarqubecloud Bot commented Jul 3, 2026

Copy link
Copy Markdown

@ShravyaKudlu ShravyaKudlu added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Jul 3, 2026
@ShravyaKudlu ShravyaKudlu marked this pull request as ready for review July 3, 2026 05:34

@sundarmachani sundarmachani left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I checked out this PR and all the functionalities and button work as intended, such as placing an order, mandatory field restrictions, making the order as received, and marking it as stocked. Considering this is mock data, it’s expected that data won’t be saved when changes are made and refreshed. Additionally, the four cards at the top track the current status, and the monthly spend also takes up the space well. It also works well in dark mode. Overall, this is a good implementation.

Image Image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants