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

Separate OSV and Kubevirt virtualization options from form #537

Merged
merged 1 commit into from
Sep 10, 2024

Conversation

nasark
Copy link
Member

@nasark nasark commented Aug 14, 2024

@@ -544,6 +534,20 @@ def self.params_for_create
}
end

def virtualization_options
Copy link
Member

Choose a reason for hiding this comment

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

.params_for_create is a class method so this has to be also

Suggested change
def virtualization_options
def self.virtualization_options

@miq-bot
Copy link
Member

miq-bot commented Aug 27, 2024

Checked commit nasark@3c1a488 with ruby 3.1.5, rubocop 1.56.3, haml-lint 0.51.0, and yamllint
1 file checked, 0 offenses detected
Everything looks fine. ⭐

@nasark nasark changed the title [WIP] Separate OSV and Kubevirt virtualization options from form Separate OSV and Kubevirt virtualization options from form Aug 29, 2024
@agrare agrare removed the wip label Aug 29, 2024
@agrare agrare merged commit c73a5bc into ManageIQ:master Sep 10, 2024
4 checks passed
@Fryguy
Copy link
Member

Fryguy commented Oct 8, 2024

Backported to radjabov in commit 2b89bdf.

commit 2b89bdfb8e4b9a0bf388943fad8c0859cf7f5584
Author: Adam Grare <adam@grare.com>
Date:   Tue Sep 10 11:37:12 2024 -0400

    Merge pull request #537 from nasark/subclass_osv_kubevirt
    
    Separate OSV and Kubevirt virtualization options from form
    
    (cherry picked from commit c73a5bcdf26ca515933adc3c2519582c0d4fc393)

Fryguy pushed a commit that referenced this pull request Oct 8, 2024
Separate OSV and Kubevirt virtualization options from form

(cherry picked from commit c73a5bc)
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