-
Notifications
You must be signed in to change notification settings - Fork 897
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
[api] Add chargebacks to the API. #3611
[api] Add chargebacks to the API. #3611
Conversation
@miq-bot remove_label wip |
@abellotti Please review |
"description" => chargeback_rate.description, | ||
"guid" => chargeback_rate.guid, | ||
"id" => chargeback_rate.id, | ||
"href" => "http://example.org/api/chargebacks/#{chargeback_rate.to_param}" |
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.
no need for full path here, you can just have "href" => chargebacks_url(chargeback_rate.to_param)
Checked commits imtayadeway@7b06e8d .. imtayadeway@796b79b with rubocop 0.32.1 and haml-lint 0.13.0 |
👍 Looks good, will merge when green. |
[api] Add chargebacks to the API.
https://trello.com/c/6j2aiL16
https://bugzilla.redhat.com/show_bug.cgi?id=1238271
cc/ @abellotti @gtanzillo