Skip to content

Fix discussions issues after room deletion and translation actions not being shown#14018

Merged
sampaiodiego merged 8 commits intodevelopfrom
fix-discussions
Apr 8, 2019
Merged

Fix discussions issues after room deletion and translation actions not being shown#14018
sampaiodiego merged 8 commits intodevelopfrom
fix-discussions

Conversation

@ggazzo
Copy link
Member

@ggazzo ggazzo commented Apr 6, 2019

Closes #13689
Closes #13965

@ggazzo ggazzo added this to the 1.0.0 milestone Apr 6, 2019
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-14018 April 6, 2019 06:37 Inactive
@ggazzo ggazzo temporarily deployed to rocket-chat-pr-14018 April 6, 2019 06:37 Inactive
@ggazzo ggazzo added the area: ui Touches the code on client side label Apr 6, 2019
@ggazzo ggazzo temporarily deployed to rocket-chat-pr-14018 April 6, 2019 17:29 Inactive
message.translations = {};
}
if (subscription && subscription.autoTranslate !== message.autoTranslateShowInverse) {
if (!!(subscription && subscription.autoTranslate) !== !!message.autoTranslateShowInverse) {
Copy link
Member

Choose a reason for hiding this comment

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

this is confusing, what is the motivation here?

if the code cannot be made clear, please add a comment describing what it is doing.

@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-14018 April 7, 2019 00:53 Inactive
@engelgabriel engelgabriel removed their assignment Apr 7, 2019
@sampaiodiego sampaiodiego changed the title [FIX] discussions issues, deletion, translation and textarea Fix discussions issues after room deletion and translation actions not being shown Apr 8, 2019
@sampaiodiego sampaiodiego merged commit 7fc7856 into develop Apr 8, 2019
@sampaiodiego sampaiodiego deleted the fix-discussions branch April 8, 2019 18:47
@rodrigok rodrigok mentioned this pull request Apr 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: ui Touches the code on client side

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments