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.

Sign up for GitHub

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

Jump to bottom

Add top-level API route for changes #519

Open
Mr0grog opened this issue Apr 24, 2019 · 0 comments
Open

Add top-level API route for changes #519

Mr0grog opened this issue Apr 24, 2019 · 0 comments
Labels
API Changes to the public API never-stale
Projects
Web Monitoring

Comments

@Mr0grog
Copy link
Member

Mr0grog commented Apr 24, 2019

You can get the changes related to a page at:

/api/v0/pages/{page ID}/changes

This should also be available at the top level:

/api/v0/changes

This requires:

  • Changes to config/routes.rb
  • Make sure app/controllers/api/v0/changes_controller.rb still functions with this
  • Add documentation to swagger.yaml

This is based on #239, which is directed at more speculative ideas that depend on this first.
@Mr0grog Mr0grog added the API Changes to the public API label Apr 24, 2019
@Mr0grog Mr0grog mentioned this issue Apr 24, 2019
Closed
@Mr0grog Mr0grog added this to New/Triage in Web Monitoring May 23, 2019
@Mr0grog Mr0grog moved this from New/Triage to Icebox in Web Monitoring May 23, 2019
@stale stale bot added the stale label Oct 21, 2019
@stale stale bot removed the stale label Oct 22, 2019
@edgi-govdata-archiving edgi-govdata-archiving deleted a comment from stale bot Oct 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
API Changes to the public API never-stale
Projects
Web Monitoring
  
Icebox
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Mr0grog