-
Notifications
You must be signed in to change notification settings - Fork 112
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
still using old retirement calls in the following files #1358
Labels
Comments
@d-m-u what should those be changed to? |
The updated API endpoint as of the last three versions is |
@d-m-u @abellotti Now that #1405 and #1406 have been merged, can this issue be closed? |
I still see one in api/examples/custom_action_service.adoc, I'll take care of it. |
@abellotti Now that #1405, #1406 and #1410 have been merged, can this issue be closed? |
Yep 👍 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
As of when this was opened, I think we're using the wrong call in at least the following places still:
I'll fix them as time allows. It's worth noting these changes should all go back to h and i.
https://github.com/ManageIQ/manageiq_docs/blob/39503bb22c58e413b42da6c056167c2166cd389b/api/reference/service_custom_actions.adoc#L338
https://github.com/ManageIQ/manageiq_docs/blob/abdf3b617f9f1233f76dfd877271cb71052aa001/api/overview/json.adoc#L250
https://github.com/ManageIQ/manageiq_docs/blob/ca32417489b7aa768e73cdeeb79b41d1548a6ddb/api/reference/services.adoc#L666
The text was updated successfully, but these errors were encountered: