-
-
Notifications
You must be signed in to change notification settings - Fork 829
Convert VoiceUserSettingsTab to TS #6340
Convert VoiceUserSettingsTab to TS #6340
Conversation
Signed-off-by: Šimon Brandner <simon.bra.ag@gmail.com>
Signed-off-by: Šimon Brandner <simon.bra.ag@gmail.com>
Signed-off-by: Šimon Brandner <simon.bra.ag@gmail.com>
Signed-off-by: Šimon Brandner <simon.bra.ag@gmail.com>
Signed-off-by: Šimon Brandner <simon.bra.ag@gmail.com>
Signed-off-by: Šimon Brandner <simon.bra.ag@gmail.com>
export default class VoiceUserSettingsTab extends React.Component<{}, IState> { | ||
constructor(props: {}) { | ||
super(props); |
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.
I have my doubts about this being right
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.
I agree with you, it doesn't exactly roll off the tongue.
After looking it up I came to the same conclusion as you, that looks correct but doesn't look right 😅
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.
Thank you for working on this! Looks good to me 👏
export default class VoiceUserSettingsTab extends React.Component<{}, IState> { | ||
constructor(props: {}) { | ||
super(props); |
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.
I agree with you, it doesn't exactly roll off the tongue.
After looking it up I came to the same conclusion as you, that looks correct but doesn't look right 😅
No description provided.