Skip to content

MA-23337: fix vk bridge type#601

Merged
pasha-nikitin-2003 merged 2 commits into
masterfrom
pavelnikitin/fix/vk-web-app-get-auth-token-types/MA-23337
Sep 2, 2025
Merged

MA-23337: fix vk bridge type#601
pasha-nikitin-2003 merged 2 commits into
masterfrom
pavelnikitin/fix/vk-web-app-get-auth-token-types/MA-23337

Conversation

@pasha-nikitin-2003

Copy link
Copy Markdown
Contributor

Закрывает #600

@pasha-nikitin-2003 pasha-nikitin-2003 self-assigned this Sep 2, 2025
@pasha-nikitin-2003 pasha-nikitin-2003 requested a review from a team as a code owner September 2, 2025 10:03
@pasha-nikitin-2003 pasha-nikitin-2003 force-pushed the pavelnikitin/fix/vk-web-app-get-auth-token-types/MA-23337 branch from e235893 to 1de007f Compare September 2, 2025 10:09
7lexik0n
7lexik0n previously approved these changes Sep 2, 2025
@pasha-nikitin-2003 pasha-nikitin-2003 merged commit 186c67a into master Sep 2, 2025
3 checks passed
scope: string;
expires: number;
status: boolean;
local_access_token?: string;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Что такое local_access_token? В каком случае он приходит и для чего используется?
В документации https://dev.vk.com/ru/bridge/VKWebAppGetAuthToken нет информации

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Изменение в документации и информация появятся чуть позже, пока мы просто синхронизируем публичные типы с нашей внутренней схемой. На данный момент это поле не может прийти в ответе для рядового пользователя.

Наличие опционального поля ведь не мешает вашей типизации?

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Не мешает
Спасибо за ответ

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.

5 participants