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

dev/core#1090 Update extendedSerializeData to use the Backbone namesp… #17855

Merged
merged 1 commit into from
Jul 16, 2020

Conversation

seamuslee001
Copy link
Contributor

…ace to fix an issue with recent versions of elementor wordpress plugin

Overview

This doesn't aim to solve everything in the dec/core#1090 however i found a similar variant of that issue in that a client of ours uses the elementor plugin and after installing the latest version of the edit profile option on an Event or contribution on line registration section broke

Before

Cannot edit a profile from the online registration section of Event Registration config

After

Can edit a profile from the online registration section of event Regsitration configuration

ping @kcristiano

…ace to fix an issue with recent versions of elementor wordpress plugin
@civibot
Copy link

civibot bot commented Jul 16, 2020

(Standard links)

@civibot civibot bot added the master label Jul 16, 2020
@kcristiano
Copy link
Member

@christianwach ping

@seamuslee001 I'll test. can you identify the version and if it's the FOSS version or the Paid one? I do have copies of both :(

@seamuslee001
Copy link
Contributor Author

Elementor v2.9.13 and non paid version

@seamuslee001
Copy link
Contributor Author

Jenkins retest this please

1 similar comment
@seamuslee001
Copy link
Contributor Author

Jenkins retest this please

@monishdeb
Copy link
Member

I have tested this patch on one of the staging site with Elementor installed. It works correctly. Approving this PR

@eileenmcnaughton
Copy link
Contributor

Too form-layer for a test. Test fail unrelated. I think this is OK given @seamuslee001 has confirmed FOSS issue & @monishdeb has tested

@seamuslee001 seamuslee001 merged commit 6da5e17 into civicrm:master Jul 16, 2020
@seamuslee001 seamuslee001 deleted the dev_core_1090 branch July 16, 2020 11:13
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.

4 participants