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

Fixes #22385 - Add cp_label to API #7166

Merged
merged 1 commit into from
Jan 24, 2018
Merged

Conversation

xprazak2
Copy link
Member

No description provided.

@theforeman-bot
Copy link

Issues: #22385

@@ -22,6 +22,7 @@ attributes :upstream_username
attributes :ostree_upstream_sync_policy, :ostree_upstream_sync_depth, :compute_ostree_upstream_sync_depth => :computed_ostree_upstream_sync_depth
attributes :deb_releases, :deb_components, :deb_architectures
attributes :ignore_global_proxy
attributes :cp_label
Copy link
Member

Choose a reason for hiding this comment

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

could we rename this to 'content_label' or something a little more descriptive?

@xprazak2
Copy link
Member Author

I renamed it to :content_label

@jlsherrill
Copy link
Member

chatting with the user that requested this, it seems that they actually wanted it in the index api, mind moving it to base.json.rabl?

@xprazak2
Copy link
Member Author

Done, I moved it into base.json.rabl

Copy link
Member

@jlsherrill jlsherrill left a comment

Choose a reason for hiding this comment

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

Thanks @xprazak2 !

@jlsherrill jlsherrill merged commit ee0fb7f into Katello:master Jan 24, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants