Skip to content

pinone: fix auto configuration (thx @joni999)#32

Merged
jsm174 merged 4 commits intomasterfrom
pinone-debug
Dec 17, 2025
Merged

pinone: fix auto configuration (thx @joni999)#32
jsm174 merged 4 commits intomasterfrom
pinone-debug

Conversation

@jsm174
Copy link
Owner

@jsm174 jsm174 commented Dec 17, 2025

This fixes issue #31 reported by @joni999.

I never had a real pinone device to test with, and I missed the \r\n.

It also fixes the StringBuilder to replace all tokens, ie just the first one.

@jsm174 jsm174 merged commit fb31322 into master Dec 17, 2025
11 checks passed
@jsm174 jsm174 deleted the pinone-debug branch December 17, 2025 12:28
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