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 Search to Currencies' drop down when adding/editing Chargeback Rate #3962

Merged

Conversation

hstastna
Copy link

@hstastna hstastna commented May 18, 2018

What:
After some talk with @lpichler, I am adding search ability for Currencies while adding/editing a Chargeback Rate.

Related task:
https://bugzilla.redhat.com/show_bug.cgi?id=1498951

Why:
There will be at least 20 currencies added in the Currencies drop down so it is good to be able to search through the available currencies.

Before:
rate_before

After:
search_after1
search_after2

Copy link
Contributor

@lpichler lpichler left a comment

Choose a reason for hiding this comment

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

LGTM from functionality side 👁 👍 :shipit:

@@ -26,7 +26,12 @@
.col-md-8
= select_tag("currency",
options_for_select(currency, @edit[:new][:currency]),
"data-miq_observe" => {:url => url}.to_json)
"data-miq_observe" => {:url => url}.to_json,
Copy link
Contributor

Choose a reason for hiding this comment

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

Can't use data-miq_observe and miqSelectPickerEvent at the same time - need to remove this line ;)

@hstastna hstastna force-pushed the Add_search_currency_chargeback_reports branch from 4c1b6ad to bca2d4a Compare May 21, 2018 19:03
@miq-bot
Copy link
Member

miq-bot commented May 21, 2018

Checked commit hstastna@bca2d4a with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
0 files checked, 0 offenses detected
Everything looks fine. ⭐

@mzazrivec mzazrivec self-assigned this May 23, 2018
@mzazrivec mzazrivec added this to the Sprint 87 Ending Jun 4, 2018 milestone May 23, 2018
@mzazrivec mzazrivec merged commit 92475b1 into ManageIQ:master May 23, 2018
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