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

Fix serialization of Track.number and Route.number #10

Closed
wants to merge 1 commit into from

Conversation

georgeto
Copy link

@georgeto georgeto commented Apr 8, 2023

The number field is of type int, but Element.text expects a string, so without the conversion the assigment fails:

TypeError: Argument must be bytes or unicode, got 'int'

The number field is of type int, but Element.text expects a string,
so without the conversion the assigment fails:
  > TypeError: Argument must be bytes or unicode, got 'int'
@sgraaf sgraaf changed the base branch from main to 0.2.x-fixes April 9, 2023 18:48
@sgraaf sgraaf changed the base branch from 0.2.x-fixes to main April 9, 2023 18:49
@sgraaf sgraaf closed this in 0648a24 Apr 9, 2023
sgraaf added a commit that referenced this pull request Apr 9, 2023
…bers, closes #10, closes #11

* ♻️ Reintroduce `__init__()` methods to all `Element`-inheriting classes

* 🐛 Fix serialization of route and track numbers

* 🔖 Bump version

* 📝 Update changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant