Member

Moves the logic of TO.to_dict in subclasses to TelegramObject itself, which was mainly dealing with sequences and converting datetimes to timestamps.

Checklist for PRs

• Added .. versionadded:: version, .. versionchanged:: version or .. deprecated:: version to the docstrings for user facing changes (for methods/class descriptions, arguments and attributes)
• Created new or adapted existing unit tests
• Documented code changes according to the CSI standard
• Added new classes & modules to the docs and all suitable __all__ s

Member

Choose a reason for hiding this comment

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

Member Author

Could you also extend the to_dict tests to make sure that the recursive setting now properly extends to the (nested) lists?

This is already tested in the classes which have those, e.g. test_to_dict in test_inlinekeyboardmarkup.

Member

Choose a reason for hiding this comment

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