-
Notifications
You must be signed in to change notification settings - Fork 1
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
MM-59373: Respecting name display preference in boards #22
base: main
Are you sure you want to change the base?
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.
LGTM
className='comment-text' | ||
dangerouslySetInnerHTML={{__html: html}} | ||
/> | ||
<Provider store={(window as any).store}> |
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.
Can you share why do we need to create a store provider here? It does't look right.
webapp/src/store/teams.ts
Outdated
@@ -87,3 +93,19 @@ export const getCurrentTeamId = (state: RootState): string => state.teams.curren | |||
export const getCurrentTeam = (state: RootState): Team|null => state.teams.current | |||
export const getFirstTeam = (state: RootState): Team|null => state.teams.allTeams[0] | |||
export const getAllTeams = (state: RootState): Team[] => state.teams.allTeams | |||
|
|||
|
|||
export const getChannelsNameMapInTeam: (state: GlobalState, teamId: string) => Record<string, Channel> = createSelector( |
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.
This can be removed as its not being used. We're using the selector directly from MM redux package.
// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved. | ||
// See LICENSE.txt for license information. | ||
|
||
export const { |
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.
You need to defined the types like how its done in user survey plugin for example - https://github.com/mattermost/mattermost-plugin-user-survey/blob/29685cc4d7ca0a2e1de196c23f6165caeacaf1b4/webapp/src/types/mattermost-webapp/index.d.ts#L10
Summary
Added support to respect name display preference from channel to boards.
Ticket Link
https://mattermost.atlassian.net/browse/MM-59373