-
Notifications
You must be signed in to change notification settings - Fork 4
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
[SRM-870] Added changes for d10. #410
Conversation
…mpatible with d10.
Drupal.behaviors.paragraphsEnhancedModalAdd = { | ||
attach: function (context) { | ||
$('.paragraphs-add-dialog-enhanced .paragraphs-add-dialog-row', context).once('add-click-handler').on('click', function (event) { | ||
console.log('I am here'); | ||
$(once('add-click-handler', '.paragraphs-add-dialog-enhanced .paragraphs-add-dialog-row', context)).on('click', function (event) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this change relates to this - https://www.drupal.org/node/3158256
Updated with develop.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Permission changes and hook cleanup looks good to me.
It's a big one. Will need another set of eyes 👀 |
Jira
https://digital-vic.atlassian.net/browse/SRM-870
Problem/Motivation
Fix
Updating module and submodules to be compatible with Drupal 10
Related PRs
Find the related PRs From the Parent ticket if necessary.
Screenshots
TODO