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.
This is also fixed in 📌 Automated Drupal 11 compatibility fixes for ics_field Needs review .
Done. Thanks for the suggestion!
Merged to 2.0.x. Thanks!
We were having the same issue and the MR solves this.
Merged to 2.0.x-dev. Thanks!
Merged to 2.0.x. Thanks!
This is possibly fixed by ✨ Remove sitemap items of disabled languages Needs review ?
I guess we can close this issue since domain_lang only supports Drupal 8 and 9.
This seems to be fixed in the 2.0.x branch. If this still occurs you can reopen or create a new issue.
Nice find! However this patch doesn't seem to fix that issue.
Does this issue still exist in the 2.0.x branch?
I think this is fixed by 🐛 DomainLanguageOverrider::__construct(): Argument #1 ($current_user) must be of type AccountProxyInterface Active
Is this still an issue?
MR no longer applies.
Merged to 2.0.x. Thanks!
Merged to 2.0.x-dev. Thanks!
Merged to 8.x-1.x. Thanks!
Merged to 8.x-1.x. Thanks!
jeroent → made their first commit to this issue’s fork.
Merged to 8.x-1.x. Thanks!
Merged to 8.x-1.x. Thanks!
Merged to 8.x-1.x. Thanks!