Skip to content

Fix Boolean handling in JSONSerialization.jsonObject(with:)#85

Merged
marcprux merged 1 commit intoskiptools:mainfrom
vincentborko:main
Jan 12, 2026
Merged

Fix Boolean handling in JSONSerialization.jsonObject(with:)#85
marcprux merged 1 commit intoskiptools:mainfrom
vincentborko:main

Conversation

@vincentborko
Copy link
Contributor

previously boolean would e.g. be uploaded as Int to firestore by default

btw: tests fail for non-US locale. Formatters use Locale.current but expected values are hardcoded for en_US.

Skip Pull Request Checklist:

  • REQUIRED: I have signed the Contributor Agreement
  • REQUIRED: I have tested my change locally with swift test
  • OPTIONAL: I have tested my change on an iOS simulator or device
  • OPTIONAL: I have tested my change on an Android emulator or device

previously boolean would e.g. be uploaded as Int to firestore by default
@cla-bot cla-bot bot added the cla-signed label Jan 12, 2026
@marcprux
Copy link
Member

Great, thanks!

@marcprux marcprux merged commit 1d82f9c into skiptools:main Jan 12, 2026
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants