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

Api change #35

Merged
merged 6 commits into from Jan 11, 2019
Merged

Api change #35

merged 6 commits into from Jan 11, 2019

Conversation

dougthor42
Copy link
Owner

This changes the API: get and add are no longer used and are instead replaced with data. This will make it so that I can eventually implement #29.

# old
GET /api/v1/get/<metric_name>
POST /api/v1/add   {json_data}

# new
GET /api/v1/data/<metric_name>    # to get data for a specific metric
POST /api/v1/data   {json_data}

Also a cleanup of add_data.sh even though that's kinda out of scope of this MR. Oops.

@dougthor42 dougthor42 merged commit e44a133 into master Jan 11, 2019
@dougthor42 dougthor42 deleted the api-change branch January 11, 2019 22:19
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.

None yet

1 participant