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

Conversation

@thegrubbsian
Copy link

No description provided.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.05%) when pulling fa8ada1 on thegrubbsian:master into 5b0af80 on dashofcode:master.

@forest
Copy link
Contributor

forest commented Dec 10, 2014

Thanks for the pull request. I'll review in the next few days.

Copy link
Contributor

Choose a reason for hiding this comment

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

@thegrubbsian why isn't the update in the context of the Story object.

project  = client.project(123456)
story = project.story(847762630) 
story.attributes = { :name => 'Jane' }
story.save

or wrap setting the attributes and saving in an update method.

story.update(:name => 'Jane')

Seems more intuitive that you would get an existing Story, change some data, and then save.

@forest
Copy link
Contributor

forest commented Jan 27, 2015

@thegrubbsian I took some of your ideas here and added them to code I already had in progress. Please see latest for how to update stories.

@forest forest closed this Jan 27, 2015
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