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

CRM-20052 get Billing Block js working as expeected on Drupal 8 as it… #9859

Merged
merged 2 commits into from
Feb 28, 2017

Conversation

seamuslee001
Copy link
Contributor

@seamuslee001 seamuslee001 commented Feb 18, 2017

… does elsewhere

Lined Issue https://issues.civicrm.org/jira/browse/CRM-20052

@jackrabbithanna
Copy link
Contributor

@seamuslee001 , @totten
I've tested this with the D8 module, and can confirm that this PR fixes the problem

@jackrabbithanna
Copy link
Contributor

@colemanw this works for me

@eileenmcnaughton
Copy link
Contributor

@seamuslee001 my read on this is that it is mostly whitespace changes + a little bit of 'add the js to the header not the default' and some weighting. If this is correct, and based on Mark's testing I would say 'OK to merge', although given the rc should really have already been cut I would merge after it is cut in order to give this that bit longer in the testing stage (esp since it is not impacting on stable CMS)

@seamuslee001
Copy link
Contributor Author

@eileenmcnaughton yes that is the case and the only other thing is adding in this

$(function() {

Which is a wrapper which is a short had for $document.ready() so this only runs once the dom has been loaded ( which it would do if it was added to the page bottom as it used to) the issue is in D8 we don't appear to have a page-bottom region to add anything to.

@eileenmcnaughton
Copy link
Contributor

Ok - let's wait for the rc to be cut & then merge this.

@eileenmcnaughton eileenmcnaughton merged commit 81898ba into civicrm:master Feb 28, 2017
monishdeb pushed a commit to monishdeb/civicrm-core that referenced this pull request May 2, 2017
CRM-20052 get Billing Block js working as expeected on Drupal 8 as it…
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.

3 participants