-
Notifications
You must be signed in to change notification settings - Fork 2.7k
MM-18157 - added loading screen to members modal #3725
MM-18157 - added loading screen to members modal #3725
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.
Thanks @jordeguevara . One small question/comment.
This issue has been automatically labelled "stale" because it hasn't had recent activity. /cc @jasonblais @hanzei |
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 looks good. Please update the import and it's good to merge. Thanks!
@@ -9,7 +9,7 @@ import * as UserAgent from 'utils/user_agent.jsx'; | |||
|
|||
import ChannelMembersDropdown from 'components/channel_members_dropdown'; | |||
import SearchableUserList from 'components/searchable_user_list/searchable_user_list_container.jsx'; | |||
|
|||
import LoadingScreen from 'components/loading_screen.jsx'; |
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.
import LoadingScreen from 'components/loading_screen.jsx'; | |
import LoadingScreen from 'components/loading_screen'; |
This component was recently migrated to typescript.
This issue has been automatically labelled "stale" because it hasn't had recent activity. /cc @jasonblais @hanzei |
Hi @jordeguevara, huge thanks for your contribution! Looks like this just needs a small change to the import, and a sync against Let me know if you have any questions! |
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.
QA will test after merge.
…d-ui-testing-for-channel-reference-autocompletion * upstream/master: (58 commits) MM-18541 - Updating header description css (mattermost#4048) [MM-19441] Fix the case of redirecting to an unavailable channel (mattermost#4070) [GH-12954] Migrate 'components/admin_console/color_setting.jsx' and associated tests to TypeScript (mattermost#4080) MM-18910 Migrate utils/emoticons to typescript (mattermost#3853) Mm 12447 Migrate 'components/get_link_modal.jsx' and associated tests to TypeScript (mattermost#3954) MM 17278 - Stop posts lists from scrolling when the emoji picker is open (mattermost#3860) MM-18157 - added loading screen to members modal (mattermost#3725) MM-19017 - Updating markdown images (mattermost#4047) MM-18956: error_pages components and tests to TS (mattermost#4050) Request team from API if invite_id is missing (mattermost#4072) MM-19678 - Restored setSelected in MultiSelectList component (mattermost#4033) [GH-12956] Migrate 'components/admin_console/banner.jsx' to TypeScript (mattermost#4075) [GH-12955] Migrate 'components/admin_console/highlight.jsx' to TypeScript (mattermost#4076) Change erroneous role='icon' to role='img' (mattermost#4051) [MM-18989] Migrate 'components/save_button.jsx' and associated tests to TypeScript (mattermost#4046) correct the height of the team sidebar when there's an announcement (mattermost#4073) Publishing storybook using circle-ci (mattermost#4037) [MM-18988] Migrate components/recent_date to TypeScript (mattermost#3883) MM-18629 - Remove default value for radio elements in interacti… (mattermost#4059) Replaced functionality in deprecated lifecycle methods (mattermost#3947) ...
* added loading screen to members modal * removed div * Fix import
* added loading screen to members modal * removed div * Fix import
Summary
This pull request addresses the issue that 'no users found' is displayed when a user goes to a channel's 'manage member' modal. It now will display a 'Loading. . .' while it is loading and only display original text if after loading there are 'no users found'
Ticket Link
Jira Ticket
Fixes mattermost/mattermost#12241
Related Pull Requests