pastorsj / blog-api

A simple api for storing users and blogs
https://pastorsj.github.io/blog-api/
MIT License
3 stars 2 forks source link

Feature/graphql wrapper #68

Closed pastorsj closed 6 years ago

pastorsj commented 6 years ago

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: #51

What is the new behavior?

We now have a graphql endpoint that will be shut down to most users

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Hopefully, there is nothing that breaks this api, but I did update several of the services to use a document based approach to updating and deleting objects (users and articles)

Other information

coveralls commented 6 years ago

Coverage Status

Coverage decreased (-2.9%) to 82.571% when pulling 03e955b9e7d4cb480d8f4351f44d0b18be4aadfe on feature/graphql-wrapper into bfaad7e3488e1273ef15a6a4ecd75993d5d55078 on master.

pastorsj commented 6 years ago

Approved

Approved with PullApprove