-
Notifications
You must be signed in to change notification settings - Fork 357
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
Do not show record's info when status of record is not completed #2801
Conversation
ff88548
to
3dfd47f
Compare
@miq-bot assign @martinpovolny |
:javascript | ||
miq_tabs_init('#rep_tabs', '/report/rep_change_tab'); | ||
- if @report.nil? | ||
= render :partial => 'layouts/info_msg', :locals => {:message => _("No Reports available.")} |
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.
3dfd47f
to
33650be
Compare
@karelhala your fix causes a regression at the queued node's parent. |
33650be
to
5848705
Compare
Checked commit karelhala@5848705 with ruby 2.3.3, rubocop 0.47.1, haml-lint 0.20.0, and yamllint 1.10.0 |
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.
Do not show record's info when status of record is not completed (cherry picked from commit 00c69cc) https://bugzilla.redhat.com/show_bug.cgi?id=1517942
Gaprindashvili backport details:
|
Fixes error when navigating to not yet finished report
When queuing new report in CI -> reports, and the report is not yet finished it will throw nasty error. This PR fixes such issue and won't allow users to do changes to such report unless it's finished.
UI changes
Before
After
BZ
https://bugzilla.redhat.com/show_bug.cgi?id=1508152