Skip to content

fix: add missing inventory status messages for clarity - #634

Merged
stdavis merged 2 commits into
mainfrom
fix/incomplete-status-message
Jul 27, 2026
Merged

stdavis merged 2 commits into
mainfrom
fix/incomplete-status-message

Conversation

@stdavis

@stdavis stdavis commented Jul 27, 2026

Copy link
Copy Markdown
Member

Related to #633

@stdavis
stdavis enabled auto-merge (rebase) July 27, 2026 17:46
@stdavis
stdavis requested a review from Copilot July 27, 2026 17:46

Copilot AI left a comment

Copy link
Copy Markdown

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 addresses issue #633 where the inventory status control could render as a blank box on the Admin Inventory Review page by adding missing UI messaging/styling for early (“draft”) inventory statuses.

Changes:

  • Add incomplete and complete status labels/messages to the whatIsNeeded mapping.
  • Render an expanded status panel for incomplete/complete states so the admin UI no longer opens to an empty panel.
  • Add button styling for incomplete and complete status states.

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

Comment thread src/api/uic-inventory/src/components/Pages/Admin/Review.jsx
auto-merge was automatically disabled July 27, 2026 17:58

Head branch was pushed to by a user without write access

@stdavis
stdavis enabled auto-merge (rebase) July 27, 2026 18:00
@stdavis
stdavis requested a review from acneville July 27, 2026 18:00
@stdavis
stdavis merged commit 4af158c into main Jul 27, 2026
7 checks passed
@stdavis
stdavis deleted the fix/incomplete-status-message branch July 27, 2026 20:25
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.

4 participants