Skip to content
This repository has been archived by the owner on Mar 13, 2024. It is now read-only.

Fixing guest being removed from other not-visible user from a channel #3914

Merged
merged 5 commits into from
Oct 11, 2019

Conversation

jespino
Copy link
Member

@jespino jespino commented Oct 9, 2019

Summary

When a guest was removed from a channel and the user wasn't visible, it needs
to ask for the user information to confirm that the other user is no longer
visible.

Ticket Link

MM-19169

@jespino jespino requested review from srkgupta and a team October 9, 2019 11:18
@jespino jespino added the 2: Dev Review Requires review by a core commiter label Oct 9, 2019
@ghost ghost requested review from mgdelacroix and migbot and removed request for a team October 9, 2019 11:18
@jespino jespino added this to the v5.16.0 milestone Oct 9, 2019
@jespino jespino added CherryPick/Approved Meant for the quality or patch release tracked in the milestone 3: QA Review Requires review by a QA tester labels Oct 9, 2019
Copy link
Contributor

@migbot migbot left a comment

Choose a reason for hiding this comment

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

Can you add a test for this scenario in websocket_actions.test.jsx?

@srkgupta srkgupta added QA Review Done and removed 3: QA Review Requires review by a QA tester labels Oct 9, 2019
@jespino
Copy link
Member Author

jespino commented Oct 10, 2019

@migbot Sure

@jespino
Copy link
Member Author

jespino commented Oct 10, 2019

Done @migbot PTAL

@jespino jespino requested a review from migbot October 10, 2019 18:02
Copy link
Contributor

@migbot migbot left a comment

Choose a reason for hiding this comment

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

Thanks @jespino!

@jespino jespino requested review from reflog and streamer45 and removed request for mgdelacroix and reflog October 11, 2019 10:00
Copy link
Contributor

@streamer45 streamer45 left a comment

Choose a reason for hiding this comment

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

Looks good! I've just left a possible suggestion.

actions/websocket_actions.jsx Outdated Show resolved Hide resolved
@jespino jespino added 4: Reviews Complete All reviewers have approved the pull request and removed 2: Dev Review Requires review by a core commiter labels Oct 11, 2019
@jespino jespino merged commit 6c0634e into mattermost:master Oct 11, 2019
@jespino jespino deleted the MM-19169 branch October 11, 2019 14:40
@mattermod mattermod added CherryPick/Done Successfully cherry-picked to the quality or patch release tracked in the milestone and removed CherryPick/Approved Meant for the quality or patch release tracked in the milestone labels Oct 11, 2019
jespino pushed a commit that referenced this pull request Oct 11, 2019
* Fixing guest being removed from other not-visible user from a channel

* Adding tests verifying the new behavior

* fixing lint errors

* Addressing PR review comments

* Fix linting error
@amyblais amyblais added Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation labels Oct 11, 2019
Wicked7000 pushed a commit to Wicked7000/mattermost-webapp that referenced this pull request Oct 12, 2019
…mattermost#3914)

* Fixing guest being removed from other not-visible user from a channel

* Adding tests verifying the new behavior

* fixing lint errors

* Addressing PR review comments

* Fix linting error
@srkgupta srkgupta added the Tests/Done Release tests have been written label Oct 14, 2019
brewsterbhg pushed a commit to brewsterbhg/mattermost-webapp that referenced this pull request Nov 11, 2019
…mattermost#3914)

* Fixing guest being removed from other not-visible user from a channel

* Adding tests verifying the new behavior

* fixing lint errors

* Addressing PR review comments

* Fix linting error
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
4: Reviews Complete All reviewers have approved the pull request Changelog/Not Needed Does not require a changelog entry CherryPick/Done Successfully cherry-picked to the quality or patch release tracked in the milestone Docs/Not Needed Does not require documentation QA Review Done Tests/Done Release tests have been written
Projects
None yet
6 participants