Skip to content
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

Feat/get organizations data #38

Merged
merged 147 commits into from
Jul 24, 2024
Merged

Feat/get organizations data #38

merged 147 commits into from
Jul 24, 2024

Conversation

kirill-ivanovvv
Copy link
Member

No description provided.

@kirill-ivanovvv kirill-ivanovvv requested a review from Nelfimov July 23, 2024 16:40
Copy link
Member

@Nelfimov Nelfimov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

stores подними в корень

Copy link
Member

@Nelfimov Nelfimov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Кнопка в disabled даже после ввода email

Screenshot 2024-07-24 at 08 19 25

Ошибка на странице организаций

Screenshot 2024-07-24 at 08 20 04

Так же - вынеси store в корень проекта. Ну и stores нейминг

@kirill-ivanovvv
Copy link
Member Author

Кнопка в disabled даже после ввода email

этой формой не занимался, думаю, она к странице users относится.
делал только страницу organizations

@kirill-ivanovvv
Copy link
Member Author

Ошибка на странице организаций

вот этот момент не понимаю. ты не менял права приложения? внизу сайдбара разместили кнопку с шестеренкой

@Nelfimov
Copy link
Member

Ошибка на странице организаций

вот этот момент не понимаю. ты не менял права приложения? внизу сайдбара разместили кнопку с шестеренкой

image

@kirill-ivanovvv
Copy link
Member Author

@Nelfimov правки внес, ревью запросил

в сафари както отличается работа с куки? сейчас у меня есть хелпер, который строку (куки) собирает в объект (куки).

добавил проверку токена при запросе организаций, должен вернуть ошибку если он будет false

webKit у себя проверил, все работает корректоно. перезаходил через оба браузера - без изменений.

@kirill-ivanovvv kirill-ivanovvv requested a review from Nelfimov July 24, 2024 07:24
@kirill-ivanovvv
Copy link
Member Author

webKit у себя проверил

image

@Nelfimov Nelfimov merged commit a434a07 into master Jul 24, 2024
3 checks passed
@kirill-ivanovvv
Copy link
Member Author

@Nelfimov организации отрисовались?

@Nelfimov
Copy link
Member

@effektivnayarabota1 да. Нужно было перелогиниться. Тут два варианта:

  • либо обновленные разрешения для приложения схватываются только после перелогина
  • либо у меня старые креды остались

@Nelfimov Nelfimov deleted the feat/get-organizations-data branch February 17, 2025 07:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants