handle deprecated endpoints

civisanalytics / civis-python

Civis API Python Client

BSD 3-Clause "New" or "Revised" License

34 stars 26 forks source link

Closed salilgupta1 closed 4 years ago

salilgupta1 commented 4 years ago

One thing to note:

We'll have code that asserts that an endpoint must have both deprecated and warning set, not one or the other.

This PR will close #345.

elsander commented 4 years ago

@stephen-hoover Wanted to flag this for you, since you might have more context on the original decision here.

@salilgupta1 , can you note in the description that this closes #345 ?

stephen-hoover commented 4 years ago

@stephen-hoover Wanted to flag this for you, since you might have more context on the original decision here.

Thanks for the flag, but I don't recall being involved in any discussions around the initial implementation of this part of the code.