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
Fixes #7493 - Store description entered during publish #4679
Conversation
@daviddavis what is the reasoning behind having description, notes and publish_description all to describe the same field? |
So the notes field holds the description for a publish or a promote. I think we could rename it to description for consistency. Thus, you would have description (the db field), publish_description, and (eventually) promote_description. |
@ehelms updated to use a consistent column (description). |
$scope.working = true; | ||
ContentView.publish(contentView, success, failure); | ||
ContentView.publish(data, success, failure); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why aren't you able to just send the contentView
object as the data here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Because contentView
has a description. What we're passing here to the API is a description for the new content view version.
ACK |
Fixes #7493 - Store description entered during publish
No description provided.