Release 4.0.0#30
Open
qpounodb wants to merge 56 commits into
Open
Conversation
- getAccessTokenParams is required in callVkApi - checkVkPlatform: void -> null - setVkViewSettings: arguments order
… Утилита для проверки на соответствие типу ошибки от ВК
checkVkAvatarIsDefault: обработка кейса с пустой строкой вместо ссылки. checkVkPlatform: добавление платформ десктопного вк-мессенджера. getVkAccessToken: обработка запроса пустого скоупа.
Функции: checkVkAvatarIsDefault, checkVkPlatform, getVkAccessToken
* Уточнение названий тестов * Оптимизация кейсов
* Обновлена версия в package.json * В readme добавлена секция про покрытие автотестами
v3.0.0: Удаление записи полей с параметрами приложения в window
SPECIAL-14044: Исправление issues
SPECIAL-14239: Подключение mediaproject-utils@5.0.0
…аметров приложения в window
…з функции объекта
Покрытие тестами
SPECIAL-14238: Добавление платформ VK мессенджера
qpounodb
commented
Jun 18, 2024
natalya-z
approved these changes
Jun 21, 2024
n-merkulova
requested changes
Oct 3, 2024
Comment on lines
+17
to
+20
| return error.error_data.error_msg; | ||
| } | ||
|
|
||
| const { error_reason } = error.error_data; |
Collaborator
There was a problem hiding this comment.
[+] мб стоит добавить проверку на наличие поля error_data? предполагаю, что могут быть казусы, когда в error будет что-то нестандартное
Comment on lines
1
to
6
| import { WindowType } from '../types/window'; | ||
|
|
||
| declare global { | ||
| // eslint-disable-next-line @typescript-eslint/no-empty-interface | ||
| interface Window extends WindowType {} | ||
| } |
Collaborator
There was a problem hiding this comment.
[+] можно избавиться от этого, если нигде не используется
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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.
По результатам объединения PRов от @dewolixgit в релиз 4.0.0 входят:
Добавления:
callVkApi,parseVkScopes,checkOneScopesSetIncludesAnother,getNewVkAccessToken,getVkAccessToken,checkVkPlatform,callVkApi,checkVkAvatarIsDefault,setVkViewSettings,shareVkStory,isVkError,checkVkUserDenied,shareVkPost,shareVkPostWithUpload, хукиusePolling,useEventSubscribeИзменения:
checkVkPlatformпроизошел переход от понятия isMvk к isWebcheckVkPlatformв перечень ВК-платформ добавлены платформы ВК МессенджераcheckVkPlatformв десктоп-платформы добавлены платформы десктопного ВК МессенджераcheckVkAvatarIsDefaultисправлена проверка на пустую строкуТакже при актуализации библиотеки сделан небольшой рефакторинг:
getVkStatus,getVkStatusesиsetVkStatusc default на такой же как у остальных утилит, для консинстентности при сборке библиотекиcallVkApiактуализировано значение по умолчанию параметра версии VK API:5.131->5.199