7digital / SevenDigital.Api.Wrapper

A fluent c# wrapper for the 7digital API
MIT License
16 stars 29 forks source link

Provide base route #186

Closed AnthonySteele closed 10 years ago

AnthonySteele commented 10 years ago

A toolkit to address https://github.com/7digital/SevenDigital.Api.Wrapper/issues/36 - "Enable using different API urls for each endpoint" May want to include all or some of this.

AnthonySteele commented 10 years ago

As discussed in #36 , The endpoint can supply the base route. Example here

Running code at this point allows the endpoint DTO to check app config files, conditionally re-route or whatever.

Changes here allow a different use - put a base uri or base uri provider code on the fluent api.

AnthonySteele commented 10 years ago

Can we hold off on merging this for a few days; I think now that the above is better; and that putting this code on the endpoint DTO is not going to be useful as the DTO will usually come in a library & the caller will be wanting to modify the base uri after the fact using Api.WithBaseUri(.... I will update with another commit unless anyone disagrees.

AnthonySteele commented 10 years ago

Don't merge this, superseded by https://github.com/7digital/SevenDigital.Api.Wrapper/pull/187