-
-
Notifications
You must be signed in to change notification settings - Fork 222
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
KeyboardButtonActionType open_photo #1522
Comments
Наверное эта задача частично решена в #1511 , но всё равно надо подумать как избавиться от таких ошибок в будущем. |
Исправлено. Нужен релиз. |
Уже есть полное решение проблемы? |
Пока не было релиза. Скоро сделаем |
@inyutin-maxim Это возможно? |
Есть 2 пути решения, ввести в перечисление значение по умолчанию когда не смогло ничего сделать, теоретически надо писать конвертер и метить все свойства с типом энама этим конвертером. Второй путь сделать все свойства с типом enum как наллабл. На вскидку так |
Нельзя, чтобы подобные ошибки были незамеченными, поэтому если оно будет null, то куда-то в лог должна записываться причина. |
Проблема в том что сериализатор не обладает логгером, как и его конвертеры |
@inyutin-maxim может тогда сделаем для таких полей аналоги с постфиксом Raw в формате string? При null будем опираться на них. |
Перенесено в #1556 |
Видимо появился какой-то новый тип кнопки (open_photo) и при получении обновлений через
api.Groups.GetBotsLongPollHistoryAsync
- всё падает с ошибкойJsonSerializationException
.Можно сделать так, чтобы вместо падения было null и где-нибудь лог ошибок?
Из-за таких ошибок парализуется работа всего бота. Он же не может игнорировать целый массив событий, а конкретное проигнорить нельзя.
The text was updated successfully, but these errors were encountered: