Skip to content

Add API router, first handler for /info#7

Merged
mbland merged 1 commit intomasterfrom
info-api
May 6, 2017
Merged

Add API router, first handler for /info#7
mbland merged 1 commit intomasterfrom
info-api

Conversation

@mbland
Copy link
Owner

@mbland mbland commented May 6, 2017

Also contains minor internal refactorings to share the error handler implementation between the API router and the other app routes.

@mbland mbland self-assigned this May 6, 2017
Also contains minor internal refactorings to share the error handler
implementation between the API router and the other app routes.
@coveralls
Copy link

coveralls commented May 6, 2017

Coverage Status

Coverage decreased (-2.6%) to 91.121% when pulling 390e296 on info-api into 563d4d0 on master.

@mbland
Copy link
Owner Author

mbland commented May 6, 2017

Ah, the coverage dropped because of lines in the errorHandler that I've added in anticipation of future API methods. Going to leave them in for now and merge anyway.

@mbland mbland merged commit 11fcea8 into master May 6, 2017
@mbland mbland deleted the info-api branch May 6, 2017 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants