Skip to content
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

Option to provide full list of users who have read a message in the MessageStatus component #2065

Open
dhendrie91 opened this issue Jul 20, 2023 · 3 comments
Labels
feature New capability needs api support requires backend support to resolve

Comments

@dhendrie91
Copy link

Motivation
Currently the message status read by count getReadByTooltipText() link only lists the first three users by name, if there is more than three users in the "read by" array the function truncates this list to "and N more". This is frustrating for admin users in our application as they view this as very important information for user accountability. As far as I am aware there is no other way to view the read by information for a particular message. This is frustrating from a developer view point since the MessageStatus link and the getReadByTooltipText() function appear to have all the required information in order to be able to display this information.

Proposed solution
If possible the MessageStatus component or top level message component should give some sort of option to either display the full list of read by users using the existing tooltip functionality, or an onclick event (with the list of read by users) which we could then extend to implement our own read by list.

Acceptance Criteria
Chat users should be able to view the full list of user names who have read a message.

@dhendrie91 dhendrie91 added the feature New capability label Jul 20, 2023
@AdamMacs
Copy link

+1

@MartinCupela
Copy link
Contributor

@dhendrie91 thank you for creating this request. What we could do, is to add an optional prop getReadByTooltipTextFn, that would be used instead of the default implementation. You could specify there your custom logic of how you wish to generate the contents of the tooltip.

Do you see any caveats to the above proposed solution?

@MartinCupela
Copy link
Contributor

This feature cannot be implemented as of today. The reason is that it is not possible to query and paginate over a list of users that have read a particular message or even channel.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New capability needs api support requires backend support to resolve
Projects
None yet
Development

No branches or pull requests

3 participants