Feature/refactor url handling

wp-net / WordPressPCL

This is a portable library for consuimg the WordPress REST-API in (almost) any C# application

MIT License

340 stars 130 forks source link

Feature/refactor url handling #285

Closed ThomasPe closed 2 years ago

ThomasPe commented 2 years ago

Updated Query Builder to use NameValueCollection
Store WordPressUrl in httpclient as BaseUrl
Pass defaultPath to HttpHelper so not every client needs to know about it
Update Test Projects to .NET 6
Change WordPressUri from string to Uri

ThomasPe commented 2 years ago

@navjot50 would like your opinion on these changes. especially the change in handling the defaultPath, think this way is a lot cleaner.

navjot50 commented 2 years ago

@ThomasPe yes, definitely cleaner as there will be less references to the default path all over. Also, the extension methods for query params give a better way of string handling. Thumbs up from my side.

ThomasPe commented 2 years ago

thanks!

ThomasPe commented 2 years ago

https://github.com/wp-net/WordPressPCL/issues/236