Skip to content

Issues/45 fix scheme mistake

Ghost User requested to merge issues/45-fix-scheme-mistake into main

Описание

Владислав в issue 45 обратил внимание, что в нашей схеме pydantic тип поля chat_id определен как int | None, при том, что в документации идет int | str | None. Было принято решение следовать максимально близко к доке.

Методам edit полю chat_id добавлен типа str, а также валидатор, который проверяет, что переданная строка может быть приведена в числу. Но по факту все приводится все равно к int, т.к. int идёт первым. Так же добавлен unit test

#45 (closed)

Чеклист к описанию MR:

  • В описании MR указана польза от изменений: кто получит пользу и почему он хочет этого
  • В описании MR указаны ссылки на соответствующие issues в GitLab, если есть
  • В описании MR приложены фрагменты переписки и/или ссылки на обсуждения, если такие велись за пределами GitLab Issues

Чеклист к коду в репозитории:

  • Новый код покрыт автотестами
  • К каждому новому автотесту в docstring описана функция и история в формате Product Flow
  • Документы Product Flow проверены и обновлены, если требуется
    • новый код совместим с !extension к старым концептам
    • новый код не влезает в !exclusion к старым концептам
    • добавлены новые !example в концепты, если требуется
  • Дока в README перечитана и обновлена, если требуется
  • Дока в CONTRIBUTING.md перечитана и обновлена, если требуется
  • Дока в sphinx_docs/source/index.rst перечитана и обновлена, если требуется
  • Обновлён файл CHANGES.md
Edited by Ghost User

Merge request reports