Skip to content

Fix hex encoding a byte string#433

Merged
153957 merged 1 commit intomasterfrom
fix-station-layout
Apr 16, 2025
Merged

Fix hex encoding a byte string#433
153957 merged 1 commit intomasterfrom
fix-station-layout

Conversation

@153957
Copy link
Copy Markdown
Member

@153957 153957 commented Apr 14, 2025

Old code lead to:
AttributeError: 'bytes' object has no attribute 'encode'.

Old code lead to:
AttributeError: 'bytes' object has no attribute 'encode'.
@153957 153957 requested review from davidfokkema and tomkooij April 14, 2025 18:21
Copy link
Copy Markdown
Member

@davidfokkema davidfokkema left a comment

Choose a reason for hiding this comment

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

Is this a string -> bytes Python 2 -> 3 leftover? Or did this change in a 3.x release? I can't really find when this changed. Good catch!

@153957
Copy link
Copy Markdown
Member Author

153957 commented Apr 16, 2025

I didn't 'catch' this :) I found this because Ricardo tried to submit a new layout, which caused an error to be reported to Sentry.

@153957 153957 merged commit dbf7f75 into master Apr 16, 2025
5 checks passed
@153957 153957 deleted the fix-station-layout branch April 16, 2025 12:58
@davidfokkema
Copy link
Copy Markdown
Member

I didn't 'catch' this :) I found this because Ricardo tried to submit a new layout, which caused an error to be reported to Sentry.

Ah, good that you've set up Sentry then! :)

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.

2 participants