-
Notifications
You must be signed in to change notification settings - Fork 1
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
DaymasS
merged 20 commits into
Always-provide-token-to-apps
from
providing-tokens-new-api
Nov 4, 2024
Merged
Providing tokens new api #263
DaymasS
merged 20 commits into
Always-provide-token-to-apps
from
providing-tokens-new-api
Nov 4, 2024
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
## Изменения Изменение запросов к 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 есть хендлер, рядом с которым написано убрать костыль. Там я внес ссылку на страницу ошибки в условный вайтлист, и все заработало. Костыль дополнился, когда-нибудь нужно это убрать :)
…oviding-tokens-new-api
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Изменения
Детали реализации
Check-List
black
иisort
для Back-End илиPrettier
для Front-End?