You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I need to:
Add a confirmation popup before pinning a message.
So That:
The user knows pinned messages are visible for everyone to see. It increases the user experience.
And as it is an extension of RocketChat, it would make it more similar and consistent.
Acceptance Criteria
Check pin status for message and give popup accordingly
on confirmation pin the message otherwise return
The text was updated successfully, but these errors were encountered:
Hey @Akshun-01 , would it be a good feature to have in embedded chat? I mean, I feel this app is mainly for integration into different platforms to streamline workflows. Adding an extra modal to confirm whether we want to pin may not be a very suitable idea in my opinion. The maintainers however will know better. You can ask them if you want.
Hey Zishan, I was also ambiguous about this issue and have been keeping it at hold for quite some time now. But I think it is a good addition, I get your point about its importance but I think if you consider a scenario where it is integrated someplace where professional discussions are taking place, there u dont want that ur messages get pinned by mistake. That would not be a good impression, although its something many would just ignore but still the fact we can't delete the pinned message so it would be a little better if we know we are pinning something.
Also if something is implemented in rocketchat, they must have thought it through.
But, yes at the end it's upto the maintainers, how they want their app to function.
Thanks for this thought! 🙌
As an EmbeddedChat developer
I need to:
Add a confirmation popup before pinning a message.
So That:
The user knows pinned messages are visible for everyone to see. It increases the user experience.
And as it is an extension of RocketChat, it would make it more similar and consistent.
Acceptance Criteria
The text was updated successfully, but these errors were encountered: