-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PR to implement copy to clipboard functionality to AAP virtual assistant #1450
Conversation
copy: { | ||
onClick: () => { | ||
if (message.actions) { | ||
message.actions.copy.isDisabled = true; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we do not need to disable the copy icon after a user clicked it once. Would you remove this line?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ack, updated and removed the disabled
check.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I left one comment. Thanks!
|
Jira Issue: https://issues.redhat.com/browse/AAP-37122
Description
PR to implement copy to clipboard functionality to AAP virtual assistant
Testing
Steps to test
An unexpected error occured
toand
feedback
exception to:npm run start
ref:

Scenarios tested
Production deployment