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 #26667 - Add pagination to policy dashboard #380
Conversation
.includes(:host) | ||
.of_policy(@policy.id) | ||
.latest | ||
.paginate(:page => params[:page], :per_page => params[:per_page]) |
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.
Layout/MultilineMethodCallIndentation: Align .paginate with ForemanOpenscap::ArfReport on line 11.
@latest_reports = ForemanOpenscap::ArfReport | ||
.includes(:host) | ||
.of_policy(@policy.id) | ||
.latest |
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.
Layout/MultilineMethodCallIndentation: Align .latest with ForemanOpenscap::ArfReport on line 11.
@@ -8,5 +8,10 @@ def prefetch_data | |||
@policy = ::ForemanOpenscap::Policy.find(params[:id]) | |||
dashboard = ForemanOpenscap::PolicyDashboard::Data.new(@policy, params[:search]) | |||
@report = dashboard.report | |||
@latest_reports = ForemanOpenscap::ArfReport | |||
.includes(:host) | |||
.of_policy(@policy.id) |
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.
Layout/MultilineMethodCallIndentation: Align .of_policy with ForemanOpenscap::ArfReport on line 11.
@@ -8,5 +8,10 @@ def prefetch_data | |||
@policy = ::ForemanOpenscap::Policy.find(params[:id]) | |||
dashboard = ForemanOpenscap::PolicyDashboard::Data.new(@policy, params[:search]) | |||
@report = dashboard.report | |||
@latest_reports = ForemanOpenscap::ArfReport | |||
.includes(:host) |
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.
Layout/MultilineMethodCallIndentation: Align .includes with ForemanOpenscap::ArfReport on line 11.
deca8f3
to
10b7f9e
Compare
[test foreman_openscap] |
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.
Thanks @xprazak2, works as expected, merging!
No description provided.