-
Notifications
You must be signed in to change notification settings - Fork 6.9k
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
Remove t from system console code #27054
Conversation
…_schemes_settings
|
||
copyID: string; | ||
copyDefault: string; | ||
title: MessageDescriptor; |
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.
A lot of these changes are replacing ID/default props with a single prop that takes a message descriptor
let convertMessage; | ||
if (toPublic) { | ||
convertMessage = ( | ||
<FormattedMarkdownMessage |
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.
There's still some things here that would be worth fixing such as removing FormattedMarkdownMessage
and fixing some weird whitespace in this string that the FormatJS ESLint plugin will complain about, but this diff is difficult enough to read as-is, so I'm not changing it yet
@@ -66,9 +67,21 @@ const GroupList = ({ | |||
<AbstractList | |||
header={<Header/>} | |||
renderRow={renderRow} | |||
emptyListText={isModeSync ? messages.emptyListModeSync : messages.emptyList} |
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 used to be done in the index file for AbstractList
which seemed really odd to me and was confusing TypeScript's ability to check the type of props, so I moved it here to make that easier
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.
Great work! Just a few nitpicks.
Summary
This is part of my mission to eradicate
t
from last week. Most of it is pretty straightforward replacement oft
withdefineMessage
/defineMessages
, and I also occasionally removed some use oflocalizeMessage
as well, but I'll comment on anything noteworthy below.Ticket Link
https://mattermost.atlassian.net/browse/MM-58324
Release Note