-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature/230 x role restriction #278
Conversation
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.
Sieht super aus, auch sehr gut getestet 👌
} | ||
|
||
private getCurrentModuleName(): string | null { | ||
const urlSegment = this.router.url |
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.
Das ist nur der Angular Teil oder? Dann finde ich dies eine gute Variante 👍
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.
Ja, genau. Alles nach dem #
Die ActivatedRoute enhält scheinbar nur in Komponenten die Infos zur Route.
}); | ||
|
||
it('should add header on open absences module', () => { | ||
// fail |
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.
Was meinst du hier damit?
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.
Ups, da hatte ich was vergessen zu löschen :)
1506784
to
a191e91
Compare
a191e91
to
a5e5027
Compare
No description provided.