fix: rename 'Delete' to 'Delete Message' in activity history (closes devguard#2116)#780
Conversation
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Updates the risk assessment feed UI copy for the delete action in the item dropdown.
Changes:
- Renamed the dropdown delete label from “Delete” to “Delete Message”.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| > | ||
| <DropdownMenuItem onSelect={(e) => e.preventDefault()}> | ||
| Delete | ||
| Delete Message |
timbastin
left a comment
timbastin
left a comment
There was a problem hiding this comment.
@arinbalyan thanks a lot for your contribution! Just a minor change. I am looking forward to merging this PR!
| > | ||
| <DropdownMenuItem onSelect={(e) => e.preventDefault()}> | ||
| Delete | ||
| Delete Message |
There was a problem hiding this comment.
I would prefer to have this as something like: "Delete Event in History"
Do you mind making that change?
There was a problem hiding this comment.
Done! Changed it to 'Delete Event in History' and pushed.
There was a problem hiding this comment.
just the naming change or something else ? because the copillot review is a little confusing above.
I have updated the name to "Delete Event in History" if that is what you wanted.
|
Thanks a lot!! |
Description
Fixes l3montree-dev/devguard#2116
Renames the misleading "Delete" label to "Delete Message" in the event activity history three-dot menu. The previous label implied deleting the VEX rule itself, when it only removes the history entry.
Changes
RiskAssessmentFeed.tsx: Changed "Delete" → "Delete Message" inRiskFeedItemdropdown menu itemChecklist