-
-
Notifications
You must be signed in to change notification settings - Fork 832
Implement the Advanced tab of new room settings #2525
Conversation
I should really just get something that does this for me prior to opening a PR
Once again, test failures are expected because of branch mismatches |
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.
Looks good, thanks! 😁
}; | ||
} | ||
|
||
componentWillMount(): void { |
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.
What's this lone type annotation-like thing doing here?
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.
Something my IDE autocompletes and I constantly forget to remove. I can remove it if wanted, and i should probably pay attention to what I'm overloading.
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.
It seems a bit random to type only one method, so I guess I'd remove it... hopefully it's configurable in your IDE?
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.
probably. I write a lot of typescript and that's probably interfering with the jsx settings a bit. I'll just pay a bit more attention to it
Fixes element-hq/element-web#8258
Part of element-hq/element-web#7606