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

Dialog editor fixes #1728

Merged
merged 2 commits into from
Jul 20, 2017
Merged

Conversation

martinpovolny
Copy link
Member

@martinpovolny martinpovolny commented Jul 20, 2017

First of follow-up fixes for the new Dialog Editor

  • fix the old dialog editor broken when adding the new one
  • minor JS style fixes

ping @skateman, @ZitaNemeckova

@romanblanco : FYI ^^
@romanblanco : there are more style issue in the new javascript. Should be no problem for you to fix it.

@miq-bot
Copy link
Member

miq-bot commented Jul 20, 2017

Checked commits martinpovolny/manageiq-ui-classic@9511537~...6978bb3 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
1 file checked, 0 offenses detected
Everything looks fine. 🍪

Copy link
Member

@skateman skateman left a comment

Choose a reason for hiding this comment

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

LGTM

@skateman
Copy link
Member

@miq-bot assign @himdel

@himdel himdel merged commit b6fa19d into ManageIQ:master Jul 20, 2017
@himdel himdel added this to the Sprint 65 Ending Jul 24, 2017 milestone Jul 20, 2017
@martinpovolny martinpovolny deleted the dialog_editor_fixes branch November 28, 2017 18:43
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