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

[DEMO] Notifications for ansible operations #450

Closed
wants to merge 3 commits into from

Conversation

skateman
Copy link
Member

Includes @Ladas's PR #398 and adds the required user_id as a second argument to the security_group_create method.

@miq-bot
Copy link
Member

miq-bot commented Aug 15, 2018

Checked commits skateman/manageiq-api@daf0b6b~...10176dd with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
2 files checked, 1 offense detected

app/controllers/api/subcollections/security_groups.rb

@Ladas
Copy link
Contributor

Ladas commented Aug 15, 2018

❤️

@Ladas
Copy link
Contributor

Ladas commented Aug 15, 2018

cc @agrare for the docs and dummy provider, can you include this? We'll need the user it for creation of the notification

@skateman
Copy link
Member Author

@Ladas first I'd like to test this, can you PM me some credentials for amazon?

@miq-bot miq-bot closed this Feb 18, 2019
@miq-bot
Copy link
Member

miq-bot commented Feb 18, 2019

This pull request has been automatically closed because it has not been updated for at least 6 months.

Feel free to reopen this pull request if these changes are still valid.

Thank you for all your contributions!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants