- 🇺🇸United States alison
- The two changes are exactly what Upgrade Status says needs to be fixed, and Upgrade Status didn't find any other issues when I ran the scans.
- After applying patch from #2 and re-running scans, Upgrade Status says this module is all set.
Thank you Project Update Bot! (but for real, thank you Drupal Rector + Upgrade Status teams!)
- @smulvih2 opened merge request.
- 🇨🇦Canada smulvih2 Canada 🍁
Opened a MR so I can make a drupaloverride for composer.
- 🇨🇦Canada smulvih2 Canada 🍁
For other that are interested in getting this going now:
"require": { "drupaloverride/ckeditor_balloonpanel": "dev-master" } ... "repositories": [ { "type": "package", "package": { "name": "drupaloverride/ckeditor_balloonpanel", "version": "dev-master", "type": "drupal-module", "dist": { "url": "https://git.drupalcode.org/api/v4/projects/project%2Fckeditor_balloonpanel/repository/archive.zip?sha=d2a04c", "type": "zip" } } } ]
- 🇺🇸United States NicholasS
Looks good, for others too you can patch dev versions of modules without having to override the repositories in #7
composer require 'drupal/ckeditor_balloonpanel:2.0.x-dev@dev'
(has to be dev version of module to patch the info.yml of a module)Then just patch as normally
"drupal/ckeditor_balloonpanel": { "Automated Drupal 10 compatibility fixes": "https://git.drupalcode.org/project/ckeditor_balloonpanel/-/merge_requests/2.diff" },
- 🇺🇸United States AaronBauman Philadelphia
is this plugin compatible and/or necessary with ckeditor 5?
is that a question for a separate thread, or can it be addressed here?
- 🇨🇦Canada RobLoach Earth
Need a namespace declaration at the top of BalloonPanelPlugin.php...
``` php
use Drupal\Core\DrupalKernel;
``` - 🇨🇦Canada RobLoach Earth
I'm happy to help push this patch up and make a release if anyone adds me as a maintainer.