Skip to content
This repository was archived by the owner on Apr 13, 2025. It is now read-only.

Conversation

@AlfonsoUceda
Copy link
Contributor

Hi,

I created a post operation and added create method for creating stories from project.

Please, give me feedback if I have to change something ;)

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.76%) when pulling 15767d1 on AlfonsoUceda:master into c7e606a on dashofcode:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.76%) when pulling 15767d1 on AlfonsoUceda:master into c7e606a on dashofcode:master.

@forest
Copy link
Contributor

forest commented Nov 7, 2014

@AlfonsoUceda thanks for the addition. Looks good. Do you mind adding a simple spec to create a story and update the readme with a simple example?

@AlfonsoUceda
Copy link
Contributor Author

Ok @forest but I need help with the spec, Do I have to create a real story? Do I have to mock the response with a hash?

Sorry I haven't done nothing like before.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.05%) when pulling 12c3cdd on AlfonsoUceda:master into c7e606a on dashofcode:master.

@AlfonsoUceda
Copy link
Contributor Author

@forest what do you think now?

@forest
Copy link
Contributor

forest commented Nov 8, 2014

👍 @AlfonsoUceda

forest added a commit that referenced this pull request Nov 8, 2014
@forest forest merged commit 9abe8ba into irphilli:master Nov 8, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants