Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Fix for PositionConverter.Read SerializerSerializer - Issue #18#19

Merged
matt-lethargic merged 1 commit intoGeoJSON-Net:mainGeoJSON-Net/GeoJSON.Text:mainfrom
mlaban:position-converter-optionsmlaban/GeoJSON.Text:position-converter-optionsCopy head branch name to clipboard
Nov 17, 2022
Merged

Fix for PositionConverter.Read SerializerSerializer - Issue #18#19
matt-lethargic merged 1 commit intoGeoJSON-Net:mainGeoJSON-Net/GeoJSON.Text:mainfrom
mlaban:position-converter-optionsmlaban/GeoJSON.Text:position-converter-optionsCopy head branch name to clipboard

Conversation

@mlaban
Copy link
Contributor

@mlaban mlaban commented Nov 16, 2022

Passing the SerializationOptions along in the internal call to JsonSerializer in PositionConverter.Read(). See issue for details and repro with floating point string literals.

…et#18

Passing the SerializationOptions along in the internal call to JsonSerializer in PositionConverter.Read(). See issue for details and repro with floating point string literals.
@mlaban mlaban closed this Nov 17, 2022
@mlaban mlaban deleted the position-converter-options branch November 17, 2022 08:42
@mlaban mlaban restored the position-converter-options branch November 17, 2022 08:44
@mlaban mlaban reopened this Nov 17, 2022
Copy link
Member

@matt-lethargic matt-lethargic left a comment

Choose a reason for hiding this comment

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

LGTM

@matt-lethargic matt-lethargic merged commit a68257e into GeoJSON-Net:main Nov 17, 2022
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.