-
Notifications
You must be signed in to change notification settings - Fork 435
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
Hide chat tab for folders #2605
Conversation
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.
Tested and works 👍
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.
nextcloud/server#18449 did not make it to beta 3 in server, so blocking merge of this pull request until the tarball of Talk for beta 3 is done.
Further changes are needed too for #2611 (besides just removing the placeholder message for folders).
Will merging this into master of talk break anything (problem) or will it just always show the tab (wouldnt mind)? |
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.
Works
It will probably break, as the constructor parameters changed (but I have not tested). |
I have tested with that branch checked out in the server @danxuliu |
Yes, what I mean is that it will break without that branch ;-) |
But isn't that in already? |
but it was merged after beta3, so if we want to have somethign working, we have to wait until beta3 compatible package has been built |
Signed-off-by: Marco Ambrosini <marcoambrosini@pm.me>
Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
Now the chat tab is not displayed for folders, so the placeholder that was shown in that case can be removed. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
0f4d73a
to
4bc3b69
Compare
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.
Rebased and adjusted to explicitly leave conversations when the chat tab can not be shown.
Based on nextcloud/server#18449
Signed-off-by: Marco Ambrosini marcoambrosini@pm.me