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] Show Favorite icon only if user has joined room #21976

Merged
merged 3 commits into from
Nov 22, 2022
Merged

[FIX] Show Favorite icon only if user has joined room #21976

merged 3 commits into from
Nov 22, 2022

Conversation

shrinish123
Copy link
Contributor

The favorite button doesn't work if a user has not joined the room or discussion and feels like a bug. Also, there is no point to show a favorite button for a channel to which the user has not joined it.

Proposed changes (including videos or screenshots)

Before: There was a favorite button even if the user hasn't joined the room

After:

favorite new

Issue(s)

Steps to test or reproduce

Go to a room which you have not joined and try to favorite the room

Further comments

@shrinish123
Copy link
Contributor Author

@dougfabris Please Review ,Thanks :)

@KevLehman KevLehman requested a review from a team June 22, 2021 17:29
@CLAassistant
Copy link

CLAassistant commented Dec 17, 2021

CLA assistant check
All committers have signed the CLA.

@ggazzo ggazzo added this to the 5.4.0 milestone Nov 2, 2022
@dionisio-bot dionisio-bot bot added stat: ready to merge PR tested and approved waiting for merge and removed stat: needs QA labels Nov 22, 2022
@ggazzo ggazzo merged commit 599992c into RocketChat:develop Nov 22, 2022
MartinSchoeler pushed a commit that referenced this pull request Nov 28, 2022
Co-authored-by: Guilherme Gazzo <guilherme@gazzo.xyz>
Co-authored-by: Guilherme Gazzo <guilhermegazzo@gmail.com>
MartinSchoeler pushed a commit that referenced this pull request Nov 28, 2022
Co-authored-by: Guilherme Gazzo <guilherme@gazzo.xyz>
Co-authored-by: Guilherme Gazzo <guilhermegazzo@gmail.com>
@ggazzo ggazzo mentioned this pull request Dec 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
communityPR stat: QA skipped stat: ready to merge PR tested and approved waiting for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants