Skip to content

feat: add notes to domain detail view#348

Merged
kevwilliams merged 4 commits intomainfrom
feat/domain-notes
Feb 28, 2026
Merged

feat: add notes to domain detail view#348
kevwilliams merged 4 commits intomainfrom
feat/domain-notes

Conversation

@kevwilliams
Copy link
Contributor

@kevwilliams kevwilliams commented Feb 28, 2026

Summary

  • Adds a Notes section to the domain detail page, allowing staff to attach freeform notes to any domain
  • Resolves note creator IDs to email addresses server-side via the IAM API so the UI displays "Added by user@example.com" instead of a numeric user ID
  • Tightens the notes list layout — removes nested Card wrappers in favour of a divider-separated list with inline author/timestamp metadata

Test plan

  • Navigate to a project domain detail page and verify the Notes section appears
  • Add a note and confirm it appears immediately with the correct author email and timestamp
  • Delete a note and confirm it is removed
  • Verify that notes created by multiple users show distinct emails
  • Verify graceful fallback (user ID shown) if the IAM lookup fails for a creator
Screencast.From.2026-02-27.16-15-59.mp4

@cla-assistant
Copy link

cla-assistant bot commented Feb 28, 2026

CLA assistant check
All committers have signed the CLA.

@cla-assistant
Copy link

cla-assistant bot commented Feb 28, 2026

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

scotwells
scotwells previously approved these changes Feb 28, 2026
@kevwilliams kevwilliams merged commit b97e378 into main Feb 28, 2026
9 checks passed
@kevwilliams kevwilliams deleted the feat/domain-notes branch February 28, 2026 00:28
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.

2 participants