Top-gg-Community / dotnet-sdk

MIT License
32 stars 11 forks source link

Add obsolete attribute for deprecated endpoint(s) #1

Closed brianebeling closed 6 years ago

brianebeling commented 6 years ago

People get exceptions from the deprecated endpoints. Removing the method would break the code of existing applications, even if they don't use the endpoint anymore.

I'd suggest a new major version indicating a major change and removing the method and adjusting the unit tests.

For now the Obsolete keyword will give them a warning.

velddev commented 6 years ago

Oh, I wasn't aware that VoterIds was obsolete, :+1: