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

Switch priceset selector to addField method #14843

Merged
merged 1 commit into from
Jul 21, 2019

Conversation

mattwire
Copy link
Contributor

Overview

On Manage Event->Amount, ContributionPage->Amount and ContributionPage->Membership switch over the price_set selector to use the addField method. I'd like to make these editable via popup like some of the other fields and this is a first step towards that.

Before

Fields added using hardcoded metadata and old add method.

After

Fields added using schema metadata and addField method.

Technical Details

Switch to metadata methods like we've done elsewhere.

Comments

@civibot
Copy link

civibot bot commented Jul 19, 2019

(Standard links)

@civibot civibot bot added the master label Jul 19, 2019
@mattwire mattwire force-pushed the priceset_to_metadatafield branch from f009116 to a0a2c91 Compare July 19, 2019 16:04
@eileenmcnaughton
Copy link
Contributor

eileenmcnaughton commented Jul 21, 2019

I tried changing price set on the contribution page fees tab & member tab along with the civievent fees tab & it all worked fine.

I note that I hit js errors on the event one until I disabled civivolunteer but I know @ginkgomzd is planning on dropping a civivolunteer release soon & I suspect I'm running the released version not the soon-to-be released version - so I think these are unrelated & pre-existing errors

@eileenmcnaughton eileenmcnaughton merged commit 3a6012d into civicrm:master Jul 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants