Skip to content

feat: add duplicate note button#2

Merged
IgnorancePulls merged 3 commits into
mainfrom
feat/duplicate-note-button
Mar 24, 2026
Merged

feat: add duplicate note button#2
IgnorancePulls merged 3 commits into
mainfrom
feat/duplicate-note-button

Conversation

@IgnorancePulls
Copy link
Copy Markdown
Owner

Summary

  • Adds a Duplicate button (document icon) to each NoteCard, appearing on hover alongside Edit and Delete
  • Clicking it creates a copy of the note with (copy) appended to the title
  • Uses the existing createNote from context — no new API calls needed

Test plan

  • Hover over a note card — duplicate button (document icon) appears to the left of Edit
  • Click duplicate — new note appears in the list with (copy) in the title
  • Original note is unchanged

🤖 Generated with Claude Code

Adds a duplicate button (document icon) that appears on hover alongside
the existing edit and delete buttons. Creates a copy of the note with
"(copy)" appended to the title.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@deepsource-io
Copy link
Copy Markdown

deepsource-io Bot commented Mar 24, 2026

DeepSource Code Review

We reviewed changes in 14bd087...ba0186e on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

Important

Some issues found as part of this review are outside of the diff in this pull request and aren't shown in the inline review comments due to GitHub's API limitations. You can see those issues on the DeepSource dashboard.

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Code Review Summary

Analyzer Status Updated (UTC) Details
JavaScript Mar 24, 2026 9:22a.m. Review ↗
Secrets Mar 24, 2026 9:22a.m. Review ↗

IgnorancePulls and others added 2 commits March 24, 2026 11:19
Updated from deprecated deepsource.io/cli to cli.deepsource.com/install
as per current documentation.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@IgnorancePulls IgnorancePulls merged commit ba0186e into main Mar 24, 2026
3 of 4 checks passed
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.

1 participant