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

Stories list #1

Merged
merged 6 commits into from
Feb 26, 2016
Merged

Stories list #1

merged 6 commits into from
Feb 26, 2016

Conversation

bigardone
Copy link
Contributor

Hi!
We were about to start coding a wrapper for Pivotal Tracker on Elixir until we found yours. I hope you don't mind if we contribute with some pull requests. I've started with the stories endpoint. If you check the ExTracker.Record.Story I have commented out the transitions and cycle_time_details keys because although they are mentioned in the API documentation as a beta feature actually they are not present in the response.
Cheers!

@forest
Copy link
Member

forest commented Feb 26, 2016

@bigardone thanks for jumping in. I'm happy to have help building it out.

I am just getting started. I looked at a number of API clients in Elixir and researched some best practices. I hope the code makes sense. Feel free to make suggestions.

forest added a commit that referenced this pull request Feb 26, 2016
@forest forest merged commit b4e1489 into dashofcode:master Feb 26, 2016
@bigardone
Copy link
Contributor Author

@forest no problem, I'm happy to help, you've already done most of the work so kudos for you :)
I neither have a lot of experience on Elixir but your code is really good structured and written so the missing features are going to be easy to implement. I will continue doing the stories path before moving to a different one.
Cheers!

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

2 participants