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

Fix: deletion of message get simultaneously deleted from the starred modal. #966

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

smritidoneria
Copy link
Contributor

@smritidoneria smritidoneria commented Feb 6, 2025

Brief Title

deletion of the starred message from the main chat interface gets deleted from the starred modal.

Fixes #965

Video/Screenshots

Screen.Recording.2025-02-06.at.11.02.33.AM.mov

PR Test Details

Note: The PR will be ready for live testing at https://rocketchat.github.io/EmbeddedChat/pulls/pr-<pr_number> after approval. Contributors are requested to replace <pr_number> with the actual PR number.

@smritidoneria smritidoneria changed the title Fix: deletion of message get simultane Fix: deletion of message get simultaneo Feb 6, 2025
@smritidoneria smritidoneria changed the title Fix: deletion of message get simultaneo Fix: deletion of message get simultaneously deleted from the starred modal. Feb 6, 2025
@Spiral-Memory Spiral-Memory added chore a subtle fix testing labels Feb 9, 2025
@@ -75,6 +83,19 @@ const FileMessage = ({ fileMessage }) => {
},
[messages, RCInstance, dispatchToastMessage]
);
useEffect(() => {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why it is here ? Related with files ?

@smritidoneria
Copy link
Contributor Author

Hey @Spiral-Memory , Actually there were similar issue related to the deletion of files from the sidebar, so I fixed that issue too in this pr only.

@smritidoneria
Copy link
Contributor Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore a subtle fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Deleted starred Message in the main Chat interface do appears in starred modal
2 participants