-
Notifications
You must be signed in to change notification settings - Fork 3
Conversation
useEffect(() => { | ||
isOAuth() |
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.
nit: функции обычно именуют с глагола, а is/has/...
используются для булевых значений
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.
Поправила
|
||
export const useServiceId = () => { | ||
const dispatch = useAppDispatch() | ||
const REDIRECT_URI = 'http://localhost:3000'; |
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.
В 9м спринте понадобится подставлять продовый или дев-адрес в зависимости от окружения. Это будет сложно сделать, если константа дублируюется в нескольких местах
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.
Когда настроим докер контейнер и впишем авторизацию в серверный рендеринг - поправлю этот момент
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.
В целом можно было бы сразу вынести общую константу, но не настаиваю
Добавила OAuth авторизацию, использовала хуки useServiceId и useOAuth:
Скриншоты/видяшка (если есть)
TBD (если есть)