-
Notifications
You must be signed in to change notification settings - Fork 140
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
added local-storage, guard, derectives, pipes, angular material, forms #99
base: master
Are you sure you want to change the base?
Conversation
public showCatalogCompany: boolean = true; | ||
public menuItems: string[] = upperCaseMenuItems; | ||
|
||
isUppercase: boolean = true; |
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.
можно добавить модификатор доступа
@@ -0,0 +1,23 @@ | |||
<div class="container" shadowHighLight> | |||
|
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.
2, 4, 6, 13,20, 22 лишние пустые строки удалите их
} else if (result === 'user') { | ||
this.userService.loginAsUser(); | ||
} else { | ||
return undefined; |
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.
Введите этот промпт в Chat GPT почему нельзя возвращать undefined? В чем отличие null и undefined? Является ли хорошей практикой если функция возвращает undefined
case 'long': | ||
return {dateStyle: 'long', timeStyle: 'long'}; | ||
default: | ||
return {dateStyle: 'medium', timeStyle: 'medium'}; |
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.
слетело форматирование
src/app/services/user.service.ts
Outdated
this.userSubject$.next({...this.user, isAdmin: false}); | ||
}; | ||
|
||
// public get isAdmin() { |
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.
удалите комментарий
|
||
@Injectable({providedIn: 'root'}) | ||
export class UsersApiService { | ||
|
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.
удалите лишние строки
telegram: Daniyar
https://t.me/the_happyy
Реализовал и добавил:
local-storage,
guard,
derectives,
pipes,
angular material,
forms