Skip to content

feat : add copy finding ID action#1126

Open
kavin553 wants to merge 2 commits into
utksh1:mainfrom
kavin553:feat-finding-copy-id
Open

feat : add copy finding ID action#1126
kavin553 wants to merge 2 commits into
utksh1:mainfrom
kavin553:feat-finding-copy-id

Conversation

@kavin553

Copy link
Copy Markdown
Contributor

Description

Added a new "Copy Finding ID" action to vulnerability/finding cards. Users can now copy a finding identifier directly to their clipboard with a single click instead of manually selecting and copying the ID.

Changes Made

  • Added a "Copy Finding ID" button in the finding details view.
  • Implemented Clipboard API support for copying finding IDs.
  • Added success feedback after a successful copy action.
  • Supported both desktop and mobile browsers.

Related Issues

Closes #1039

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update

How Has This Been Tested?

  1. Started the frontend application locally.
  2. Opened the Findings page.
  3. Selected multiple findings with different IDs.
  4. Clicked the "Copy Finding ID" button.
  5. Pasted the copied value into a text editor.
  6. Verified that the pasted value matched the displayed finding ID.
  7. Verified success feedback appears after copying.
  8. Tested functionality in desktop and mobile browser views.

Checklist

  • My code follows the code style of this project.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings.

@kavin553

Copy link
Copy Markdown
Contributor Author

Hi maintainers 👋

I've completed the implementation for issue #1039.

What was added

  • One-click "Copy Finding ID" action
  • Clipboard API integration
  • Success feedback after copy
  • Desktop and mobile browser support

Testing

  • Verified the copied value matches the displayed Finding ID
  • Tested multiple findings
  • Confirmed success feedback appears correctly

I would appreciate your review and feedback. Thank you for your time and support! 🚀

@utksh1 utksh1 added level:beginner 20 pts difficulty label for small beginner-friendly PRs type:feature Feature work category bonus label area:frontend Frontend React/UI work labels Jun 20, 2026

@utksh1 utksh1 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Requesting changes. This adds two separate Copy ID buttons, has inconsistent indentation, and has no tests for clipboard success/failure states. Please keep one clear action location, align with the existing Copy Brief pattern, and add focused frontend coverage.

@kavin553

Copy link
Copy Markdown
Contributor Author

Hi maintainer 👋

Addressed the requested review feedback:

✅ Kept a single Copy Finding ID action
✅ Aligned behavior with the existing Copy Brief pattern
✅ Improved success/failure handling
✅ Added focused frontend test coverage

Verification:

  • npm test passed
  • All existing tests remain green
  • Functionality verified manually

Looking forward to your review. Thank you! 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:frontend Frontend React/UI work level:beginner 20 pts difficulty label for small beginner-friendly PRs type:feature Feature work category bonus label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEAT] Add "Copy Finding ID" action on vulnerability cards

2 participants