AlienDwarf / open-meteo-dotnet

A .Net library for the Open-Meteo.com API.
MIT License
28 stars 9 forks source link

Error 400 Bad Request with culture with decimal seperator #19

Closed sbiaudet closed 1 year ago

sbiaudet commented 1 year ago

When you use openmeteo.dotnet with a culture configured with a decimal separator the request is not correctly build and open-meteo server response an error 400.

I've tested it with French Culture "fr-FR"

AlienDwarf commented 1 year ago

Thanks for the report!

InvariantCulture is by far the better approach since it's not associated with any regional settings.

sbiaudet commented 1 year ago

You're welcome !!

I've seen some others possible improvements. I'll back to you with more PR ;)