Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

adding an option to flag an api route as deprecated #268

Merged
merged 1 commit into from
Jul 29, 2014
Merged

adding an option to flag an api route as deprecated #268

merged 1 commit into from
Jul 29, 2014

Conversation

komidore64
Copy link
Contributor

if this flag is present, it adds an annotation to the route in the
api documentation

@komidore64
Copy link
Contributor Author

deprecated annotation example

@iNecas
Copy link
Member

iNecas commented Jul 25, 2014

Nice! Would you mind adding some tests around that into https://github.com/Apipie/apipie-rails/blob/master/spec/lib/method_description_spec.rb#L14 (just a simple one, that makes sure the deprecated keyword gets into the json)

@mtparet
Copy link
Contributor

mtparet commented Jul 25, 2014

👍
until now I write it in the description

if this flag is present, it adds an annotation to the route in the
api documentation
@komidore64
Copy link
Contributor Author

@iNecas test added

@iNecas
Copy link
Member

iNecas commented Jul 29, 2014

Thanks!

iNecas added a commit that referenced this pull request Jul 29, 2014
adding an option to flag an api route as deprecated
@iNecas iNecas merged commit 2cb6b35 into Apipie:master Jul 29, 2014
@komidore64 komidore64 deleted the deprecated-route-flag branch July 29, 2014 14:37
@iNecas
Copy link
Member

iNecas commented Aug 8, 2014

The changes were released as part of the 0.2.3 release. Thanks for your contribution! Let the code be with you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants