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

Moved common code to a method to reset tier details when switching Type. #7157

Conversation

h-kataria
Copy link
Contributor

This fixes an issue when user tries to change Rate type on a Add Chargeback Rate form, with the fix it should reload form to show Storage/Compute fields when type is changed.

before
99291a54-0fa9-446d-bd78-dbf6381a0770

ef777849-6db3-4a15-93e2-71586536c794

after
Screenshot from 2020-06-29 11-38-17

Screenshot from 2020-06-29 11-38-30

This fixes an issue when user tries to change Rate type on a Add Chargeback Rate form, with the fix it should reload form to show Storage/Compute fields when type is changed.
@miq-bot
Copy link
Member

miq-bot commented Jun 29, 2020

Checked commit h-kataria@492bfd9 with ruby 2.5.7, rubocop 0.69.0, haml-lint 0.28.0, and yamllint
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.

The Seal of Approval

@skateman skateman merged commit 65660b1 into ManageIQ:master Jun 29, 2020
simaishi pushed a commit that referenced this pull request Jun 29, 2020
…ching_type_on_new_record_form

Moved common code to a method to reset tier details when switching Type.

(cherry picked from commit 65660b1)
@simaishi
Copy link
Contributor

Jansa backport details:

$ git log -1
commit 6337f04f6ab1627d4e8965409c54031c4a444243
Author: Halász Dávid <[email protected]>
Date:   Mon Jun 29 18:16:27 2020 +0200

    Merge pull request #7157 from h-kataria/reload_tier_details_when_switching_type_on_new_record_form

    Moved common code to a method to reset tier details when switching Type.

    (cherry picked from commit 65660b180155d058e41c4123120cf9007c5e768a)

@h-kataria h-kataria deleted the reload_tier_details_when_switching_type_on_new_record_form branch January 18, 2021 21:47
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