I have committed proposed changes and created a merge request.
sahilgidwani → created an issue.
Adding updated patch for 1.17 version
Adding patch for v10.1.x
Adding patch for v10.1
I am also getting this issue.
Can anyone help me here with how to migrate existing ckeditor div manager from CK4 to CK5 on a existing site.
- After applying the patch mentioned in #163 ✨ Drupal 10 & CKEditor 5 readiness Fixed and clearing the cache, I am getting the following error.
Error: Call to undefined method Drupal\entity_embed\Plugin\EmbedType\Entity::getModulePath() in Drupal\entity_embed\Plugin\EmbedType\Entity->getDefaultIconUrl() (line 243 of modules/contrib/entity_embed/src/Plugin/EmbedType/Entity.php).
- And also, as mentioned in #162 ✨ Drupal 10 & CKEditor 5 readiness Fixed , the PR is not merged yet, so reopening this issue.
- Correct me if I am wrong somewhere in my understanding.
- I am getting the error for PHP Stan command failed, as mentioned in the issue description, but not getting any error related to "No files found to analyse".
- When I debugged the analyze method of the DeprecationAnalyzer class, I found that the Process class is not returning output on line 373, and as a result, we are getting null in $json.
- Further, the condition check implemented for checking any JSON errors is returning true, and it is showing errors, although the PHP stan command is being executed successfully without any errors.
Anyone can please look into this and implement a solution to resolve the issue mentioned above.
MR is not merged because of error, we need to reopen this issue.
Added patch file for the diff file added by @codebymilkey
Please review the patch added in comment 17
We need to remove the core key also in info.yml, removed the core key and update the patch.
sahilgidwani → created an issue.
I am reopening this issue, as I am getting warnings and errors, as mentioned in the descriptions in version 9.5.9 and PHP 8.
Not able to apply patch, therefore adding new patch.
sahilgidwani → created an issue.
Rerolled patch for drupal core 9.5.x
Rerolled patch 12 for Drupal 9.5
Any update on this issue? I am also facing this issue in my production environment but not on Pantheon.
Please try to follow the MR workflow instead of patches for new issues.
I have reviewed the PR and it looks perfect to me.
Moving the issue to RTBC.
I reviewed the PR and it looks perfect to me.
Moving it to RTBC.
The issue is a duplicate of
3336589
✨
Drupal 10 support
Fixed
.
Therefore closing it as duplicate.
Not able to reproduce the issue on my local D10 commerce environment. Can you please add the details of which version of commerce module you are using and also drupal core version.
I have made the required changes and checked on D10, it is working fine. Also, I have updated the readme.md according to the latest standards also.
Moving it to needs review.
I am working on this issue.
I have searched for the hook_page_attachments occurrences throughout the D10 core and created a new patch for D10, also changed version of this issue to 10.1.x.
Moving it to needs review.
I will reroll the patch.
Also, @smustgrave should we change the issue version to D10?
I have checked and reviewed MR and it works perfectly for me.
Moving it to RTBC.
I have checked and reviewed MR and it works perfectly for me.
Moving it to RTBC.
Mr looks good to me.
Moving to RTBC.
I will review patch and will update the status once done.
Also, please try to follow the MR workflow for new issues.
MR looks perfect.