Closed jankapunkt closed 1 month ago
Describe the bug
_startLoggingIn() { this._reactiveDict.set('_loggingIn', true); Data.notify('loggingIn'); }, _startLoggingOut() { User._isLoggingOut = true; Data.notify('loggingOut'); }, _endLoggingIn() { this._reactiveDict.set('_loggingIn', false); Data.notify('loggingIn'); }, _endLoggingOut() { User._isLoggingOut = false; Data.notify('loggingOut'); },
it's not determinable, at which method the even has fired as they are clearly the same.
Expected behavior It should be distinguishable at which function the events fired. Proposed change would be to use clear naming:
_startLoggingIn() { this._reactiveDict.set('_loggingIn', true); Data.notify('loggingIn'); }, _startLoggingOut() { User._isLoggingOut = true; Data.notify('loggingOut'); }, _endLoggingIn() { this._reactiveDict.set('_loggingIn', false); Data.notify('loggedIn'); }, _endLoggingOut() { User._isLoggingOut = false; Data.notify('loggedOut') },
Closing this issue due to no activity. Feel free to reopen.
Describe the bug
it's not determinable, at which method the even has fired as they are clearly the same.
Expected behavior It should be distinguishable at which function the events fired. Proposed change would be to use clear naming: