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

Removing ems_events from config/api.yml #14699

Merged
merged 1 commit into from
Apr 7, 2017

Conversation

abellotti
Copy link
Member

Removing ems_events from config/api.yml

collection definition specified but no implementation behind
it giving us a 404 for it and messing up the client.

collection definition specified but no implementation behind
it giving us a 404 for it and messing up the client.
@abellotti
Copy link
Member Author

/cc @gtanzillo @imtayadeway this gotta go. Thanks.

@abellotti
Copy link
Member Author

/cc @mkanoor thanks for noticing this.

@miq-bot
Copy link
Member

miq-bot commented Apr 7, 2017

Checked commit abellotti@c4e5c79 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
0 files checked, 0 offenses detected
Everything looks good. 🏆

Copy link
Member

@gtanzillo gtanzillo left a comment

Choose a reason for hiding this comment

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

👍

@gtanzillo gtanzillo added this to the Sprint 58 Ending Apr 10, 2017 milestone Apr 7, 2017
@gtanzillo gtanzillo merged commit a374504 into ManageIQ:master Apr 7, 2017
simaishi pushed a commit that referenced this pull request Apr 11, 2017
Removing ems_events from config/api.yml
(cherry picked from commit a374504)
@simaishi
Copy link
Contributor

Fine backport details:

$ git log -1
commit bd0899d9dde84434b2976906b89632e7d658202e
Author: Gregg Tanzillo <[email protected]>
Date:   Fri Apr 7 18:03:07 2017 -0400

    Merge pull request #14699 from abellotti/api_remove_invalid_ems_events
    
    Removing ems_events from config/api.yml
    (cherry picked from commit a3745049a8876598fa9e8fdc84adad0cb34120e9)

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.

4 participants