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

Ansible Tower Job Template shows the previously selected record on return #12066

Merged

Conversation

lgalis
Copy link
Contributor

@lgalis lgalis commented Oct 19, 2016

Ansible Tower Job Template shows the previously selected record when returning from a different view

Links

https://bugzilla.redhat.com/show_bug.cgi?id=1382616

@lgalis
Copy link
Contributor Author

lgalis commented Oct 19, 2016

@miq_bot add_label wip

@lgalis
Copy link
Contributor Author

lgalis commented Oct 19, 2016

@miq-bot add_label ui, bug

@lgalis
Copy link
Contributor Author

lgalis commented Oct 19, 2016

@miq-bot add_label wip

@lgalis lgalis changed the title Ansible Tower Job Template shows the previously selected record on return [WIP] Ansible Tower Job Template shows the previously selected record on return Oct 19, 2016
@miq-bot miq-bot added the wip label Oct 19, 2016
@lgalis lgalis force-pushed the ansible_job_template_display_on_return_to_tab branch from 013468a to 98380ee Compare October 20, 2016 21:57
@lgalis lgalis changed the title [WIP] Ansible Tower Job Template shows the previously selected record on return Ansible Tower Job Template shows the previously selected record on return Oct 20, 2016
@lgalis
Copy link
Contributor Author

lgalis commented Oct 20, 2016

@miq-bot remove_label wip

before do
right_cell_text = nil
login_as user_with_feature(%w(providers_accord configured_systems_filter_accord configuration_scripts_accord))
controller.instance_variable_set(:@right_cell_text, right_cell_text)
Copy link
Member

Choose a reason for hiding this comment

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

why not the simpler: controller.instance_variable_set(:@right_cell_text, nil)?

@lgalis lgalis force-pushed the ansible_job_template_display_on_return_to_tab branch from 98380ee to dc7c4aa Compare October 21, 2016 02:47
@lgalis lgalis force-pushed the ansible_job_template_display_on_return_to_tab branch from dc7c4aa to 5059c74 Compare October 21, 2016 02:54
@miq-bot
Copy link
Member

miq-bot commented Oct 21, 2016

Checked commits lgalis/manageiq@43a54c3~...5059c74 with ruby 2.2.5, rubocop 0.37.2, and haml-lint 0.16.1
1 file checked, 0 offenses detected
Everything looks good. 🍪

@mzazrivec mzazrivec self-assigned this Oct 21, 2016
@mzazrivec mzazrivec added this to the Sprint 48 Ending Oct 24, 2016 milestone Oct 21, 2016
@mzazrivec mzazrivec merged commit 0e7133d into ManageIQ:master Oct 21, 2016
@lgalis lgalis deleted the ansible_job_template_display_on_return_to_tab branch October 21, 2016 12:54
@lgalis lgalis restored the ansible_job_template_display_on_return_to_tab branch October 21, 2016 19:51
@chessbyte
Copy link
Member

chessbyte pushed a commit that referenced this pull request Oct 21, 2016
…_return_to_tab

Ansible Tower Job Template shows the previously selected record on return
(cherry picked from commit 0e7133d)

https://bugzilla.redhat.com/show_bug.cgi?id=1387708
@chessbyte
Copy link
Member

Euwe Backport details:

$ git log -1
commit d8075246e2da0b9f2c9d9b9ec7ae3000137058c1
Author: Milan Zázrivec <mzazrivec@redhat.com>
Date:   Fri Oct 21 13:34:15 2016 +0200

    Merge pull request #12066 from lgalis/ansible_job_template_display_on_return_to_tab

    Ansible Tower Job Template shows the previously selected record on return
    (cherry picked from commit 0e7133db403b7d9273ba42f1431ca11829e048bc)

    https://bugzilla.redhat.com/show_bug.cgi?id=1387708

@lgalis lgalis deleted the ansible_job_template_display_on_return_to_tab branch October 22, 2016 22:18
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.

None yet

5 participants