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 puffs endpoint #29

Merged
merged 5 commits into from
Jul 9, 2018
Merged

add puffs endpoint #29

merged 5 commits into from
Jul 9, 2018

Conversation

nandub
Copy link
Contributor

@nandub nandub commented Jul 5, 2018 •
edited
Loading

A work in progress implementation for the puffs endpoint. To test it in the back end use http://localhost:3000/api/v1/puffs

This pull will add into #15 and closes #28

@nandub nandub force-pushed the feature/puffs-endpoint branch from 2b5e10a to 5ee65af Compare July 5, 2018 19:49
@nandub nandub mentioned this pull request Jul 6, 2018
Closed
SamiUrias
SamiUrias reviewed Jul 9, 2018
View reviewed changes
Copy link
Contributor

@SamiUrias SamiUrias left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This endpoint works well,
I test the creation of a puff and the read of all puffs.

I approve the merge

@nandub nandub merged commit 4627c46 into development Jul 9, 2018
@nandub nandub deleted the feature/puffs-endpoint branch July 9, 2018 03:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SamiUrias SamiUrias SamiUrias left review comments

@ChandanaChaitanya ChandanaChaitanya Awaiting requested review from ChandanaChaitanya

Assignees
No one assigned
Labels
in progress review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Puff Endpoint
2 participants
@nandub @SamiUrias