-
-
Notifications
You must be signed in to change notification settings - Fork 833
GET /teams from RTS instead of config.json #658
Conversation
Now that the RTS contains config for teams, use GET /teams to get that information so that users will see be able to register as a team (but not yet auto-join rooms, be sent to welcome page or be tracked as a referral).
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.
Should we wait until we've fetched the teams config before showing the registration form?
|
||
if (this.props.teamServerConfig && | ||
this.props.teamServerConfig.teamServerURL && | ||
!this._rtsClient) { |
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.
Our usual indenting style here would be:
if (
this.props.teamServerConfig &&
[...]
) {
}
We could make the component |
Probably ought to |
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.
Otherwise lgtm
}).finally(() => { | ||
this.setState({ | ||
teamServerBusy: false, | ||
}); |
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 still want a .done() on the end to catch stray exceptions.
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.
oh ok, thanks
@@ -297,6 +307,8 @@ module.exports = React.createClass({ | |||
}, | |||
|
|||
_getRegisterContentJsx: function() { | |||
var Spinner = sdk.getComponent("elements.Spinner"); |
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.
Also const please
Now that the RTS contains config for teams, use GET /teams to get that information so that users will see be able to register as a team (but not yet auto-join rooms, be sent to welcome page or be tracked as a referral).