- Issue created by @bohart
- First commit to issue fork.
- Assigned to kksandr
- Merge request !18fix: Bump up minimal Drupal version of module as well as php version. Removed... β (Closed) created by Unnamed author
- Issue was unassigned.
- Status changed to Needs review
9 months ago 5:51pm 28 June 2024 I fixed the schema issues and removed unused keys in the schema.
I also fixed other tests that did not pass, this is out of scope, so if this is a problem I can remove it.
- π³πΏNew Zealand Josh Waihi
I can't save Text Format configuration forms in the admin UI in Drupal 10.3 with this module enabled. Changing the schema from config_object to mapping (as this MR did) seemed to fix the problem.
I don't think I understand why this MR wasn't merged?
- πΊπΈUnited States SocialNicheGuru
There are two different MRs. which to use?
- Status changed to Needs work
17 days ago 2:07pm 17 March 2025 - πΊπ¦Ukraine dinazaur
The schema bug should be fixed in the latest dev. I keep this ticket open because we need to remove not existing options from config schema e.g.
use_advanced_pattern: type: boolean label: 'Use advanced pattern?' pattern: type: string label: 'Pattern' advanced_pattern: type: string label: 'Advanced pattern'
- πΊπΈUnited States SocialNicheGuru
Does that take an update hook in addition to removing them in config?
- πΊπ¦Ukraine dinazaur
I didn't test it but from my understanding after updating to CKEditor5 obsolete options are removed automatically, someone needs to check it out.