7digital / SevenDigital.Api.Wrapper

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

Is IFluentApi.ClearParameters() still useful? #206

Closed AnthonySteele closed 10 years ago

AnthonySteele commented 10 years ago

Should we remove IFluentApi.ClearParameters() ?

It looks like an invitation to use an IFluentApi instance for a second request. This is a pattern that we prefer to discourage - you should use a new instance for each request as mentioned in the "instancing " section here: https://github.com/7digital/SevenDigital.Api.Wrapper/blob/master/ReleaseNotes40.md

It is not used in the wrapper project itself. But because it's public, it could be in (ab)use.

AnthonySteele commented 10 years ago

Removed in 5.0.0