Skip to content

Commit

Permalink
Merge pull request #43 from dougthor42/remove-confusing-route
Browse files Browse the repository at this point in the history
Remove the confusing /api/v1/<metric> route.
  • Loading branch information
dougthor42 committed Jan 18, 2019
2 parents 4071a43 + 372cb96 commit 6f18516
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 1 deletion.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@


## Unreleased
+ Removed a confusing route: `/api/v1/<metric>`. (#39)
+ Added a title that will link back to the index page.


Expand Down
1 change: 0 additions & 1 deletion src/trendlines/routes.py
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,6 @@ def post_datapoint():
return msg, 201


@api.route("/api/v1/<metric>", methods=["GET"])
@api.route("/api/v1/data/<metric>", methods=["GET"])
def get_data_as_json(metric):
"""
Expand Down

0 comments on commit 6f18516

Please sign in to comment.