-
Notifications
You must be signed in to change notification settings - Fork 78
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: activate community channel link #12653
Conversation
Jenkins BuildsClick to see older builds (7)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM :)
Just noted that it doesn't work for not yet spectated/joined communities. Fixing. |
dfb1c53
to
0b1e419
Compare
@borismelnik @MishkaRogachev I've updated the PR, please check it out again 🙂 Now switching to a channel of non-spectated/joined community works as well. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
0b1e419
to
67832f9
Compare
Fixes #12539
What does the PR do
chatID
for chat lookup, as it should be a concatenation of community id and channel idactivateStatusDeepLink
. Channel should be processed earlier, because communityId is also setchannelUuid
fieldswitchTo
functionScreenshot of functionality (including design for comparison)
Screen.Recording.2023-11-08.at.11.51.32.mov