-
-
Notifications
You must be signed in to change notification settings - Fork 829
Fail more softly on homeserver liveliness errors #3067
Conversation
This performs liveliness checks on the auth pages to try and show a friendlier error. Earlier checks in the app startup are expected to not block the app from loading on such failures. See element-hq/element-web#9828
This and the partner PRs are blocked on a proper design for the error. |
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 I think this is basically there now
@@ -447,6 +467,7 @@ module.exports = React.createClass({ | |||
onEditServerDetailsClick={onEditServerDetailsClick} | |||
flows={this.state.flows} | |||
serverConfig={this.props.serverConfig} | |||
canSubmit={this.state.serverIsAlive} |
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.
same here for not preventing form submission without an obvious way to re-check (for registration I guess you could toggle server-type but the user wouldn't know to do that).
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.
Registration has a different phases cycle from the other 3, so I'm not convinced I need to fix the form here. There is no button to click if the server is down, regardless of liveliness checks: it's just a spinner.
But still check so we can clear any errors, maybe
Fixes element-hq/element-web#9828
Requires matrix-org/matrix-js-sdk#938
Requires element-hq/element-web#9960
This performs liveliness checks on the auth pages to try and show a friendlier error. Earlier checks in the app startup are expected to not block the app from loading on such failures.