Skip to content

Commit 67110b5

Browse files
MartinCarpentierMartin Boje Carpentier
andauthored
Feature/update readme (#157)
* Create dotnet.yml * Updated the readme to point to geojson.text for system.text.json support Co-authored-by: Martin Boje Carpentier <mbc@kamstrup.com>
1 parent 4a10d5b commit 67110b5

1 file changed

Lines changed: 6 additions & 4 deletions

File tree

README.md

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -28,15 +28,17 @@ Point point = JsonConvert.DeserializeObject<Point>(json);
2828

2929
See the [Tests](https://github.com/GeoJSON-Net/GeoJSON.Net/tree/master/src/GeoJSON.Net.Tests) for more examples.
3030

31-
## Special considerations for ASP.Net Core 3+
31+
## Special considerations for ASP.Net Core 3.1+
3232

33-
System.Text.Json is the default (recommended) serializer. GeoJSON.Net **does not play well with System.Text.Json**. Here are the recommended steps as a workaround:
33+
System.Text.Json is the default (recommended) serializer. GeoJSON.Net **does not play well with System.Text.Json** and only supports Newtonsoft.Json
34+
35+
[GeoJSON.Text](https://github.com/GeoJSON-Net/GeoJSON.Text) is a fork of this repository and have been developed to support System.Text.Json.
36+
37+
If you still need to use Newtonsoft.Json in ASP.Net Core 3.1+, you must overwrite the default serializer.
3438

3539
- add the "Microsoft.AspNetCore.Mvc.NewtonsoftJson" NuGet Package
3640
- add "services.AddControllers().AddNewtonsoftJson();" to your service configuration.
3741

38-
(Thanks for @jrowe88 for pointing this out)
39-
4042
## Contributing
4143
Highly welcome! Just fork away and send a pull request. We try and review most pull requests within a couple of days. There is now a version 2.0.0 branch. I've created this ready for any breaking changes and any extra features and would encourage anything that isn't a bug fix to go in there.
4244

0 commit comments

Comments
 (0)