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

MM-20180 Fix search in channel members modal #4242

Merged
merged 2 commits into from
Nov 22, 2019

Conversation

Pomyk
Copy link
Contributor

@Pomyk Pomyk commented Nov 15, 2019

Summary

Loading screen was shown on each search and that messed up the input. Fixed by displaying loading screen only on startup.

Ticket Link

https://mattermost.atlassian.net/browse/MM-20180

Display loading screen only on startup.
Fixes mattermost/mattermost#13095
@hanzei hanzei added 2: Dev Review Requires review by a core commiter 3: QA Review Requires review by a QA tester labels Nov 18, 2019
@hanzei hanzei modified the milestones: v5.19.0, v5.18.0 Nov 18, 2019
@amyblais amyblais added the CherryPick/Approved Meant for the quality or patch release tracked in the milestone label Nov 18, 2019
Copy link
Member

@saturninoabril saturninoabril left a comment

Choose a reason for hiding this comment

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

Tested and LGTM, thanks @Pomyk!

@saturninoabril saturninoabril added QA Review Done and removed 3: QA Review Requires review by a QA tester labels Nov 19, 2019
@hmhealey
Copy link
Member

/update-branch

@amyblais amyblais added 4: Reviews Complete All reviewers have approved the pull request and removed 2: Dev Review Requires review by a core commiter labels Nov 22, 2019
@hmhealey hmhealey merged commit 4aaddb7 into mattermost:master Nov 22, 2019
@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 Nov 22, 2019
@amyblais amyblais added Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation labels Nov 22, 2019
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
Projects
None yet
7 participants