-
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
Remove Timelines tab from Report editor #6181
Conversation
if @tl_changed # Reload the screen if the timeline data was changed | ||
page.replace_html("tl_sample_div", :partial => "form_tl_sample") if @tl_field != ReportHelper::NOTHING_STRING | ||
elsif @formatting_changed # Reload the screen if the formatting pulldowns need to be reset | ||
if @formatting_changed # Reload the screen if the formatting pulldowns need to be reset | ||
page.replace_html("formatting_div", :partial => "form_formatting") | ||
elsif @tl_repaint |
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.
@bmclaughlin should this go away too?
@bmclaughlin can you also review following, i think those can be removed too |
41e0b52
to
efbcfb7
Compare
Checked commits bmclaughlin/manageiq-ui-classic@fddc085~...efbcfb7 with ruby 2.4.6, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0 spec/controllers/miq_report_controller/reports/editor_spec.rb
|
Thanks for the review @h-kataria! I removed the lines you mentioned above and ran into no errors unit testing and running the specs locally. 👍 |
Remove Timelines tab from Report editor (cherry picked from commit a58ce1a) Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1784183
Ivanchuk backport details:
|
Before:
After:
@miq-bot add_labels bug
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1743651