Conversation
Ebert has finished reviewing this Pull Request and has found:
But beware that this branch is 1 commit behind the You can see more details about this review at https://ebertapp.io/github/justarrived/just-match-web/pulls/390. |
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.
@bex1 A W E S O M E! Absolutely awesome! ❤️
import {Resolve} from '@angular/router'; | ||
import {User} from '../../models/user'; | ||
import {UserProxy} from '../../services/proxy/user-proxy.service'; | ||
|
||
@Injectable() | ||
export class UserResolver implements Resolve<User> { | ||
private readonly storageAuthorizationData: string = 'authorizationData'; | ||
private readonly storageSessionKey: string = 'sessionData'; |
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.
fyi this will logout all users on deploy
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.
Yes, might want to change the string back, but I thought session was a better name 👍
@buren
🔪 User manager and auth manager.
User now resolved via resolver before init if there is a session.
Also added loading indicators on various components before data load etc.
When the language is changed and logged in -> user is reloaded ->whole page is dimmed with a loading indicator until user is resolved again.