feat(devops-copilot): add copy button on each message - #2867
Conversation
|
View your CI Pipeline Execution ↗ for commit 55fe1e7
💡 Verify your cache is correct by running tasks in a sandbox. Read docs ↗ ☁️ Nx Cloud last updated this comment at |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## staging #2867 +/- ##
===========================================
- Coverage 49.58% 48.49% -1.10%
===========================================
Files 850 853 +3
Lines 21162 21356 +194
Branches 6375 6434 +59
===========================================
- Hits 10494 10356 -138
- Misses 8898 9260 +362
+ Partials 1770 1740 -30
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
| } | ||
|
|
||
| function VoteButtons({ messageId, threadId }: { messageId: string; threadId?: string }) { | ||
| function CopyRichTextButton({ |
There was a problem hiding this comment.
You need to use CopyButton from the design system or use similar code, hook, buttons etc
There was a problem hiding this comment.
Since the CopyButton copies in plain text, I couldn't use it as-is, so I reused the button's exact components; if it's cleaner, I can modify the component to allow a choice between plain and rich text @RemiBonnet
There was a problem hiding this comment.
Ah yes, I didn’t realize that rich text requires a different Clipboard API and that our current hook only supports plain text. Since this is specific to the rendered Copilot message and you already reuse the design-system components, your approach makes sense. No need to change the shared hook or CopyButton just for this use case, go for it 👍 (You can keep without the wording it's much cleaner)
|
🎉 This PR is included in version 1.337.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Summary
Issue: QOV-2141
Screenshots / Recordings
Testing
yarn testoryarn test -u(if you need to regenerate snapshots)yarn formatyarn lintPR Checklist
.cursor/rules)feat(service): add new Terraform service) - required for semantic-release