- Issue created by @prashant.c
- 🇮🇳India prashant.c Dharamshala
The root cause behind this is when there is no "Media type" defined in the
/admin/structure/media
but instead of the error in the page, can we handle it on theText format
configuration page itself while saving the configurations with "Embed Media" enabled. - First commit to issue fork.
- Merge request !11939Added validation message, used contructor property promotion → (Open) created by prashant.c
- 🇮🇳India prashant.c Dharamshala
No status message appears in the functional test after "Save configuration", which has the text
The text format ckeditor5 has been updated.
However, this status message is being displayed correctly on manual testing.
Based on this, I removed the code
$assert_session->pageTextContains('The text format ckeditor5 has been updated');
If there is some alternative to this, please suggest.
Thanks
- 🇮🇳India prashant.c Dharamshala
The issue mentioned in #7 🐛 Text formats "Embed Media" filter throws error Needs work was related to validation error itself. Added validation to check if the Medie embed filter is enabled.