Merge pull request #2058 from matrix-org/t3chguy/hide_logout_warning_whilst_logging_out
while logging out ignore `Session.logged_out` as it is intentionalpull/21833/head
						commit
						07083e61b7
					
				|  | @ -386,6 +386,8 @@ function _persistCredentialsToLocalStorage(credentials) { | |||
|     console.log(`Session persisted for ${credentials.userId}`); | ||||
| } | ||||
| 
 | ||||
| let _isLoggingOut = false; | ||||
| 
 | ||||
| /** | ||||
|  * Logs the current session out and transitions to the logged-out state | ||||
|  */ | ||||
|  | @ -405,6 +407,7 @@ export function logout() { | |||
|         return; | ||||
|     } | ||||
| 
 | ||||
|     _isLoggingOut = true; | ||||
|     MatrixClientPeg.get().logout().then(onLoggedOut, | ||||
|         (err) => { | ||||
|             // Just throwing an error here is going to be very unhelpful
 | ||||
|  | @ -420,6 +423,10 @@ export function logout() { | |||
|     ).done(); | ||||
| } | ||||
| 
 | ||||
| export function isLoggingOut() { | ||||
|     return _isLoggingOut; | ||||
| } | ||||
| 
 | ||||
| /** | ||||
|  * Starts the matrix client and all other react-sdk services that | ||||
|  * listen for events while a session is logged in. | ||||
|  | @ -451,6 +458,7 @@ async function startMatrixClient() { | |||
|  * storage. Used after a session has been logged out. | ||||
|  */ | ||||
| export function onLoggedOut() { | ||||
|     _isLoggingOut = false; | ||||
|     stopMatrixClient(); | ||||
|     _clearStorage().done(); | ||||
|     dis.dispatch({action: 'on_logged_out'}); | ||||
|  |  | |||
|  | @ -1262,6 +1262,7 @@ export default React.createClass({ | |||
|             }, true); | ||||
|         }); | ||||
|         cli.on('Session.logged_out', function(call) { | ||||
|             if (Lifecycle.isLoggingOut()) return; | ||||
|             const ErrorDialog = sdk.getComponent("dialogs.ErrorDialog"); | ||||
|             Modal.createTrackedDialog('Signed out', '', ErrorDialog, { | ||||
|                 title: _t('Signed Out'), | ||||
|  |  | |||
		Loading…
	
		Reference in New Issue
	
	 Bruno Windels
						Bruno Windels