-
Notifications
You must be signed in to change notification settings - Fork 10.4k
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
[FIX] Double output of message actions #11902
Conversation
Actions markup is output twice, in immediate succession: https://github.com/RocketChat/Rocket.Chat/blob/135c6f8e1bb204cd7b07682dec8e189144fe3871/packages/rocketchat-message-attachments/client/messageAttachment.html#L152-L158 @ubarsaiyan and @ggazzo - Please review and get this merged asap. Did no one even look at this feature before it went in release?
@timkinnane I noticed it when it was deployed and sent a PR asap but got no response. #11853 |
Right. Thanks @ubarsaiyan I searched issues for "actions" but not buttons. Better to be specific, there's a lot of buttons in Rocket.Chat. |
@timkinnane Sorry, I thought this would be merged quickly so didn't give much thought in the title. I will keep this is in mind from now onwards. :) |
@ubarsaiyan was your PR merged? Can I close this? |
Actions markup is output twice, in immediate succession: https://github.com/RocketChat/Rocket.Chat/blob/135c6f8e1bb204cd7b07682dec8e189144fe3871/packages/rocketchat-message-attachments/client/messageAttachment.html#L152-L158 @ubarsaiyan and @ggazzo - Please review and get this merged asap. Did no one even look at this feature before it went in release?
Actions markup is output twice, in immediate succession: https://github.com/RocketChat/Rocket.Chat/blob/135c6f8e1bb204cd7b07682dec8e189144fe3871/packages/rocketchat-message-attachments/client/messageAttachment.html#L152-L158 @ubarsaiyan and @ggazzo - Please review and get this merged asap. Did no one even look at this feature before it went in release?
Actions markup is output twice, in immediate succession:
Rocket.Chat/packages/rocketchat-message-attachments/client/messageAttachment.html
Lines 152 to 162 in 135c6f8
@ubarsaiyan and @ggazzo - Please review and get this merged asap. Did no one even look at this feature before it got merged?