Skip to content

Add ability to edit comments on cards - #61

Merged
disnet merged 1 commit into
mainfrom
card/3mf6bsqq2pn2l
Mar 7, 2026
Merged

Add ability to edit comments on cards#61
disnet merged 1 commit into
mainfrom
card/3mf6bsqq2pn2l

Conversation

@disnet

@disnet disnet commented Feb 19, 2026

Copy link
Copy Markdown
Owner

Summary

  • Adds updatedAt optional field to the comment lexicon, data model (Comment/CommentRecord), and appview schema
  • Adds updateComment() function in src/lib/comments.ts that updates text + updatedAt and triggers sync
  • Updates commentToRecord() to include updatedAt when present
  • Adds inline edit UI in TaskEditModal: edit/cancel buttons appear on hover for own comments
  • Shows "(edited)" indicator with tooltip showing edit time when a comment has been edited
  • Updates appview: schema migration for existing databases, upsertComment includes updatedAt, firehose processor passes updatedAt through
  • Frontend appview.ts loads updatedAt from the appview response

Test plan

  • Create a comment on a card, verify it appears normally
  • Hover over own comment, verify edit (pencil) and delete buttons appear
  • Click edit, verify textarea shows current comment text
  • Edit text and click Save, verify comment updates and shows "(edited)"
  • Hover over "(edited)" to verify tooltip shows edit timestamp
  • Click Cancel during edit, verify original text is preserved
  • Verify other users' comments do not show edit button
  • Verify comment editing syncs to PDS (check sync status dot)
  • Verify npm run check passes with 0 errors
  • Verify npm run build succeeds

🤖 Generated with Claude Code

Sprite @claude
Add ability to edit comments on cards
0dc2395 
Users can now edit their own comments via an inline edit UI. Adds
updatedAt field to the comment data model, an updateComment() function,
and edit/cancel buttons in the task modal. Edited comments show an
"(edited)" indicator with tooltip showing edit time.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@disnet
disnet merged commit 165b75c into main Mar 7, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@disnet