Fixed the PHPCS errors and added a basic test to check if the IndexPreCreate event is triggered.
Added the event class and dispatch it before the index is created.
Instead of adding a dependency to the domain module, why don't we use
Drupal\Core\Routing\LocalRedirectResponse that is included in core?
Module is Drupal 11 compatible, so I guess we can close this issue.
I can confirm that MR 179 fixes the problem for me.
Had the same issue. Currently, I can't install the module on Drupal 11.
@vasike, I tried an initial version, which is now available in the MR.
If you could give that one a try.
This issues seems to be fixed by issue
📌
Use formUpdated event instead of change event
Needs review
.
Can you confirm @sander wemagine?
The MR i proposed, updates the button to this:
Still not ideal but it works.
I fixed the failing tests.
There are still some tests failing on PHP 8.5, but I see the same failurs on the 11.2.x branch:
https://git.drupalcode.org/project/drupal/-/pipelines/614551
So I think those are not related tot his issue.
@kewin.nammert, you're right. This problem was caused by a custom module we use.
Thank you!
Thank you!
I contacted ajay_reddy → and vuil → .
jeroent → made their first commit to this issue’s fork.
I just created a 3.0.4 release.
I updated the tests and those are green again.
Closed 🐛 Errors '"onomasticon_XXXX" is not a supported key.' when add or update text format Active as a duplicate.
I can confirm this bug and the MR fixed the problem in my case.
Done. Thanks for the reminder!
Merged to dev. Thanks!
Merged to dev. Thanks!
jeroent → made their first commit to this issue’s fork.
jeroent → made their first commit to this issue’s fork.
Merged in dev. Thanks!
We are also using this patch on a project. However, in Elastic Cloud, we are using a serverless project.
This patch was working fine for us, with some extra changes.
In BackendClient, we got errors on the /_close and /_open API calls, so we just had to disable those.
But perhaps this is not the right place and this should be handled in a follow-up issue.
I can confirm this bug and wrote a MR to fix this.
jeroent → made their first commit to this issue’s fork.
Made a MR of the patch and added testing to check if HTML is stripped from the summary field.
This issue seems to be working fine. I added a basic test case to check if location is added to the .ical file.
This is still active. However, the uploaded patch only adds a prefix for the datetime module. Not for token.
jeroent → changed the visibility of the branch 3312088-php-error to hidden.
This is fixed in the latest versions.