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

Игнорировать лишние параметры в JSON request от Алисы #9

Closed
LionZXY opened this issue Feb 16, 2020 · 2 comments
Labels
bug Something isn't working enhancement New feature or request

Comments

@LionZXY
Copy link

LionZXY commented Feb 16, 2020

Основная мысль

Предлагаю во всех объектах добавить игнорирование дополнительных параметров при парсинге JSON

Мотивация

  1. API в основном расширяется, а не меняется. Это значит что с большой вероятностью при обновлении API навык будет продолжать работать
  2. Поддержка дебажных JSON с полями типо __comment
  3. Постепенная миграция со старой версии библиотеки на новую (не будет крашиться)

В связи со спецификой разработки навыков (обычно это пет-проекты) при ближайшем расширении API протокола некоторая часть навыков просто умрет навсегда

В результате

  • Повышение стабильности работы навыков на библиотеке aioalice (1)
  • Поддержка API основанных на API алисы (2,3)
@mahenzon
Copy link
Owner

Есть пара идей, что с этим сделать, в отдельной ветке поэксперементирую, уже есть наработки

@mahenzon
Copy link
Owner

@mahenzon mahenzon added bug Something isn't working enhancement New feature or request labels Feb 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants