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

Providing tokens new api #263

Merged
merged 20 commits into from
Nov 4, 2024

Conversation

BatuevIO
Copy link
Contributor

@BatuevIO BatuevIO commented Nov 4, 2024

Изменения

Детали реализации

Check-List

  • Вы проверили свой код перед отправкой запроса?
  • Вы написали тесты к реализованным функциям?
  • Вы не забыли применить форматирование black и isort для Back-End или Prettier для Front-End?

## Изменения
Изменение запросов к api на apiClient из api-uilib. 

## Детали реализации
- Запросы теперь делаются через методы apiClient с соответствующими
путями и параметрами.
- Типы данных для переменных берутся из components в библиотеке
api-uilib.
## Изменения
Фикс ошибок, возникших при изменении апи на apiClient.

## Детали реализации

- Изменил условие проверки стора приложений, чтобы совершался запрос.
## Изменения
Починил отображение методов входа оаутх.

## Детали реализации

- Проверялось неправильное условие
## Изменения
Добавил логи для оаутха, чтобы понять, почему они не обновляются. Убрал
условие на проверку стора.

## Детали реализации
<!-- Здесь можно описать технические детали по пунктам. -->
## Изменения
Изменил формат authButtons с учетом разных форм записи authMethods на
фронте и апи.
## Изменения
Добавил лог для привязки и отвязки кнопок.
## Изменения
<!-- Опишите здесь на языке, понятном каждому, изменения, сделанные в
исходном коде по пунктам. -->

## Детали реализации
<!-- Здесь можно описать технические детали по пунктам. -->

## Check-List
<!-- После сохранения у следующих полей появятся галочки, которые нужно
проставить мышкой -->
- [ ] Вы проверили свой код перед отправкой запроса?
- [ ] Вы написали тесты к реализованным функциям?
- [ ] Вы не забыли применить форматирование `black` и `isort` для
_Back-End_ или `Prettier` для _Front-End_?
## Изменения
<!-- Опишите здесь на языке, понятном каждому, изменения, сделанные в
исходном коде по пунктам. -->

## Детали реализации
<!-- Здесь можно описать технические детали по пунктам. -->

## Check-List
<!-- После сохранения у следующих полей появятся галочки, которые нужно
проставить мышкой -->
- [ ] Вы проверили свой код перед отправкой запроса?
- [ ] Вы написали тесты к реализованным функциям?
- [ ] Вы не забыли применить форматирование `black` и `isort` для
_Back-End_ или `Prettier` для _Front-End_?
## Изменения
Нашел ошибку в router/auth.ts, теперь должно быть можно залогиниться с
помощью оаутх
## Изменения
<!-- Опишите здесь на языке, понятном каждому, изменения, сделанные в
исходном коде по пунктам. -->

## Детали реализации
<!-- Здесь можно описать технические детали по пунктам. -->

## Check-List
<!-- После сохранения у следующих полей появятся галочки, которые нужно
проставить мышкой -->
- [ ] Вы проверили свой код перед отправкой запроса?
- [ ] Вы написали тесты к реализованным функциям?
- [ ] Вы не забыли применить форматирование `black` и `isort` для
_Back-End_ или `Prettier` для _Front-End_?
…i-uilib (#247)

…i-uilib

## Изменения
<!-- Опишите здесь на языке, понятном каждому, изменения, сделанные в
исходном коде по пунктам. -->

## Детали реализации
<!-- Здесь можно описать технические детали по пунктам. -->

## Check-List
<!-- После сохранения у следующих полей появятся галочки, которые нужно
проставить мышкой -->
- [ ] Вы проверили свой код перед отправкой запроса?
- [ ] Вы написали тесты к реализованным функциям?
- [ ] Вы не забыли применить форматирование `black` и `isort` для
_Back-End_ или `Prettier` для _Front-End_?
## Изменения
<!-- Опишите здесь на языке, понятном каждому, изменения, сделанные в
исходном коде по пунктам. -->

## Детали реализации
<!-- Здесь можно описать технические детали по пунктам. -->

## Check-List
<!-- После сохранения у следующих полей появятся галочки, которые нужно
проставить мышкой -->
- [ ] Вы проверили свой код перед отправкой запроса?
- [ ] Вы написали тесты к реализованным функциям?
- [ ] Вы не забыли применить форматирование `black` и `isort` для
_Back-End_ или `Prettier` для _Front-End_?
## Изменения
<!-- Опишите здесь на языке, понятном каждому, изменения, сделанные в
исходном коде по пунктам. -->

## Детали реализации
<!-- Здесь можно описать технические детали по пунктам. -->

## Check-List
<!-- После сохранения у следующих полей появятся галочки, которые нужно
проставить мышкой -->
- [ ] Вы проверили свой код перед отправкой запроса?
- [ ] Вы написали тесты к реализованным функциям?
- [ ] Вы не забыли применить форматирование `black` и `isort` для
_Back-End_ или `Prettier` для _Front-End_?
## Изменения
Отменил коммиты для апи, которые выехали в прод
## Изменения
Добавил кнопку и страницу для удаления аккаунта.
## Изменения
Починил условие проверки токена пользователя в роутере auth
## Изменения
Вернул apiClient, вмержил изменения из мэйна, пофиксил чуток багов
## Изменения
Починил отображение страницы ошибки при возникновении ошибки в оаутх.

## Детали реализации
- В router/index.ts есть хендлер, рядом с которым написано убрать
костыль. Там я внес ссылку на страницу ошибки в условный вайтлист, и все
заработало. Костыль дополнился, когда-нибудь нужно это убрать :)
@BatuevIO BatuevIO marked this pull request as ready for review November 4, 2024 19:27
@BatuevIO BatuevIO requested a review from DaymasS November 4, 2024 19:27
@DaymasS DaymasS merged commit 76d1e40 into Always-provide-token-to-apps Nov 4, 2024
2 checks passed
@DaymasS DaymasS deleted the providing-tokens-new-api branch November 4, 2024 20:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants