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

Bug: Pinning a message does not shows it as a pinned message. #647

Open
Barrylimarti opened this issue Oct 30, 2024 · 3 comments
Open

Bug: Pinning a message does not shows it as a pinned message. #647

Barrylimarti opened this issue Oct 30, 2024 · 3 comments
Labels
bug Something isn't working

Comments

@Barrylimarti
Copy link
Contributor

Description:

As a user with pinning permissions, if I pinned a message in the chat, the message does not show the pinned icon immediately and also, it doesn't appear in the pinned message sidebar. However, the opposite behavior of unpinning the message is working immediately.
Also, to note that these feature do work for starring the messages.

Steps to reproduce:

  1. Type a message or use any previous message.
  2. Open the pinned messages sidebar from top right corner menu.
  3. Try pinning the message.
  4. Message gets pinned but :
    -is not shown in the menu
    -pin icon is still outline
    -pinned icon is not visible beside the message

Expected behavior:

-should be shown in the sidebar menu
-pin icon should be solid indicating pinned
-pinned icon should be visible beside the message

Actual behavior:

eroor.mov
@Barrylimarti Barrylimarti added the bug Something isn't working label Oct 30, 2024
@Barrylimarti
Copy link
Contributor Author

From my investigation,
After clicking pin button, the api does get called to pin the message and the response also, comes as success. But the messages from useMessageStore doesn't update the original message with the updated pinned flag.
Uploading a console log of messages when pin is clicked for a message. It can be seen that the original object does not change with the pinned key.

Screenshot 2024-10-31 at 12 42 59 AM

But this somehow happens perfectly for starring the message. Can anyone help me solving this issue?

@smritidoneria
Copy link
Contributor

hey @Barrylimarti , The same issue has been mentioned in #653. The pr has been raised to solve the same. You can check the pr #654.

@Barrylimarti
Copy link
Contributor Author

hey @Barrylimarti , The same issue has been mentioned in #653. The pr has been raised to solve the same. You can check the pr #654.

Sure, although there was no need to create a new issue if this was already present. I will check the PR once.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants