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

Add support for load_values_on_init to text boxes. #17814

Merged
merged 3 commits into from
Aug 10, 2018
Merged

Add support for load_values_on_init to text boxes. #17814

merged 3 commits into from
Aug 10, 2018

Conversation

jeffmcutter
Copy link
Contributor

@jeffmcutter jeffmcutter commented Aug 7, 2018

Add support for dynamic text and text area boxes for load_values_on_init to be respected when show refresh button is true.
This allows dynamic text boxes to not load on init similar to dialog_field_sorted_item used by drop downs and radio buttons.

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

Add support for dynamic text and text area boxes for load_values_on_init to be respected when show refresh button is true.
This allows dynamic text boxes to not load on init similar to dialog_field_sorted_item used by drop downs and radio buttons.
@gmcculloug gmcculloug self-assigned this Aug 9, 2018
@gmcculloug gmcculloug requested a review from eclarizio August 9, 2018 12:55
Copy link
Member

@eclarizio eclarizio left a comment

Choose a reason for hiding this comment

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

Can you add a spec for this? Otherwise looks fine. 👍

@miq-bot
Copy link
Member

miq-bot commented Aug 9, 2018

Checked commits jeffmcutter/manageiq@b241c16~...ac15dfd with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
2 files checked, 0 offenses detected
Everything looks fine. 🏆

Copy link
Member

@eclarizio eclarizio left a comment

Choose a reason for hiding this comment

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

Thanks for adding the spec 👍

@gmcculloug gmcculloug added the bug label Aug 10, 2018
@gmcculloug
Copy link
Member

@jeffmcutter I'm marking this as a bug. If you feel it needs to be back-ported to the gaprindashvili branch can you please open a BZ and we'll associate it to these changes.

@gmcculloug gmcculloug merged commit 43a9146 into ManageIQ:master Aug 10, 2018
@gmcculloug gmcculloug added this to the Sprint 92 Ending Aug 13, 2018 milestone Aug 10, 2018
@jeffmcutter
Copy link
Contributor Author

@gmcculloug I have BZ 1613987 open for this issue, will that do the trick? I definitely need it back-ported, so if not let me know I can do another one.

Thanks!

@gmcculloug
Copy link
Member

That's what I needed, thanks.

simaishi pushed a commit that referenced this pull request Sep 11, 2018
…pport_for_text_boxes

Add support for load_values_on_init to text boxes.
(cherry picked from commit 43a9146)

https://bugzilla.redhat.com/show_bug.cgi?id=1623560
@simaishi
Copy link
Contributor

Gaprindashvili backport details:

$ git log -1
commit 5741f3ec29700e988c78bd8fdb04f1dd9a5f5d26
Author: Greg McCullough <[email protected]>
Date:   Fri Aug 10 14:44:03 2018 -0400

    Merge pull request #17814 from jeffmcutter/add_load_values_on_init_support_for_text_boxes
    
    Add support for load_values_on_init to text boxes.
    (cherry picked from commit 43a91465d9ce7743b70f075b3946f4b7aaef1d75)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1623560

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.

5 participants