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
Fixes #23699 - Remediation Support for Host Status #6167
Fixes #23699 - Remediation Support for Host Status #6167
Conversation
Issues: #23699 |
Sorry for two PRs but first PR automatically closed while Github was down. |
Hello, @ares I am having one more doubt regarding taking this method to status.rb file. I moved this method as like to_label but is it Ok to make the method remediation_help_text as self method in build_status.rb file and should we pass any parameter to this method? as the method is now defined in both status.rb and build_status.rb file returning nil and string respectively. But now there is no object of class from build_status.rb that I can access in view, so should I make this method a class method? |
db11e63
to
179e949
Compare
app/models/host_status/status.rb
Outdated
@@ -72,6 +72,10 @@ def update_timestamp | |||
def update_status | |||
self.status = to_status | |||
end | |||
|
|||
def remediation_help_text | |||
return nil |
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.
Style/RedundantReturn: Redundant return detected.
a474886
to
5780973
Compare
5780973
to
b377dca
Compare
b377dca
to
dd81355
Compare
Closing in favor of GH-6692, thanks @akshay-mendhe ! |
No description provided.