derrick/ai-wave-02
1
0
Fork 0
Code Issues 4 Pull Requests 17 Actions Packages Projects 1 Releases Wiki Activity

Feature Internal Order Notes #12

Merged
derrick merged 3 commits from feat/dg-2-internal-order-notes into master 2026-09-25 03:47:37 +00:00
Conversation 0 Commits 3 Files Changed 10
+134 -3
derrick commented 2026-09-24 14:13:14 +00:00
Owner
Copy Link
Copy Source

Summary

Operations can now add internal notes to an order.

Ticket / Assignment

  • Ticket: #9

What changed

  • New notes field on orders schema
  • New order notes component + crud for adding and updating the new order notes field

How to test

   bun test apps/api/src/orders/order-notes-assignment-2.test.ts

Manual checks:

  • Go to http://localhost:5173/orders/order_1 or any orders page
  • Enter internal notes, below shipping address form.
  • Click Save notes.
  • Confirm success toast and saved notes remain visible.

Screenshots or recordings

image.png

Risks / follow-ups

  • No known follow-ups.

PR checklist

  • change is scoped to one ticket or one small reviewable concern
  • acceptance criteria were checked
  • code follows CONTRIBUTING.md
  • no unrelated refactor is mixed in
  • debug code and temporary logs are removed
  • relevant automated checks were run locally
  • changed behavior was manually tested when needed
  • known risks or follow-ups are called out
## Summary Operations can now add internal notes to an order. ## Ticket / Assignment - Ticket: https://git.letphil.com/derrick/ai-wave-02/issues/9 ## What changed - New `notes` field on orders schema - New order notes component + crud for adding and updating the new order notes field ## How to test ```bash bun test apps/api/src/orders/order-notes-assignment-2.test.ts ``` Manual checks: - Go to http://localhost:5173/orders/order_1 or any orders page - Enter internal notes, below shipping address form. - Click Save notes. - Confirm success toast and saved notes remain visible. ## Screenshots or recordings ![image.png](/attachments/9f2410ad-c21a-4b8a-8442-8dac0339c331) ## Risks / follow-ups - No known follow-ups. ## PR checklist - [x] change is scoped to one ticket or one small reviewable concern - [x] acceptance criteria were checked - [x] code follows `CONTRIBUTING.md` - [x] no unrelated refactor is mixed in - [x] debug code and temporary logs are removed - [x] relevant automated checks were run locally - [x] changed behavior was manually tested when needed - [x] known risks or follow-ups are called out
image.png
23 KiB
derrick added 3 commits 2026-09-24 14:13:14 +00:00
add:notes column to orders 2e2694354d
add:order notes crud 97438aaa56
add:order notes ui component
Runner Test / test (push) Successful in 17s
Details
Runner Test / test (pull_request) Successful in 16s
Details
698a49ae82
derrick merged commit 8235ceb61f into master 2026-09-25 03:47:37 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
ahmedhnoor aziz_28002 cq0622 dsel karinah0333_16363 notsicilian riso8739 derrick
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: derrick/ai-wave-02#12