richardgaunt → created an issue.
@thisisalistairsaccount
We have a long running PR to fix this but there is no easy solution. At the moment, CivicTheme works out which menu to render based on the menu name.
We have not been able to get to a good solution on how to configure within Drupal (within the theme system) a menu block to render a menu with a certain theme.
richardgaunt → created an issue.
richardgaunt → created an issue.
richardgaunt → created an issue.
I am all good I made a local patch to fix for my purposes.
https://git.drupalcode.org/issue/external_entities-3516546/-/jobs/4830977
Tests are failing with this change but won't fix until I know the direction to take.
Created a MR that fixes this issue:
https://git.drupalcode.org/project/external_entities/-/merge_requests/81
Not sure whether inheritance should be this way as ConfigurableExternalEntityTypeInterface feels like an extension on ExternalEntityTypeInterface but its a starting point.
richardgaunt → created an issue.
richardgaunt → created an issue.
richardgaunt → created an issue.
fionamorrison23 → credited richardgaunt → .
richardgaunt → created an issue.
Thanks I think we have solved this issue.
We are now developing directly on Drupal 11 and we have added patches for any modules that CivicTheme relies on in composer to make them Drupal 11 compatible.
These fixes are being released with CivicTheme 1.10 which is being released today.
Hi,
We have created some recipes to get CivicTheme started in Drupal CMS. These will activate and setup CivicTheme with some default content to get you started:
https://github.com/civictheme/upgrade-tools
Here is the PR with explanation
https://github.com/civictheme/upgrade-tools/pull/3
This will work with the upcoming release of Civictheme 1.10.
Fixes have been implemented and we are now developing on D11 in our development environment. Marking this as fixed for upcoming release.
We have created a repository for this update script (details being outlined in the release notes of 1.10)
PR for this update script and screenshots
https://github.com/civictheme/upgrade-tools/pull/1
Repository to get this script from: https://github.com/civictheme/upgrade-tools
We are adding a recipe to enable on Drupal CMS: https://github.com/civictheme/upgrade-tools/pull/3 please see the drupal-cms recipes directory.
We have solutions for these upcoming in the release being prepared and scheduled for tomorrow. There will be an alpha recipe for installing on Drupal CMS and we are now developing on Drupal 11.
We are using the following patches for incompatible modules. To install these module dependencies on Drupal 11 add these patches to your composer.json:
"drupal/simple_sitemap": {
"Error on installing / enabling simple_sitemap in D11: https://www.drupal.org/project/simple_sitemap/issues/3507718": "./patches/modules/simple_sitemap/fix-error-on-install.patch"
},
"drupal/styleguide": {
"Create Drupal 11 test release: https://www.drupal.org/project/styleguide/issues/3505640": "https://git.drupalcode.org/project/styleguide/-/merge_requests/23.patch"
}PR created and merged: https://github.com/civictheme/monorepo-drupal/pull/1359
richardgaunt → created an issue.
richardgaunt → created an issue.
jct321 → credited richardgaunt → .
richardgaunt → created an issue.
richardgaunt → created an issue.
Only test required is the administration tabs ("View, Edit and delete") still render with correct active state on Front-end of the website
Added fix to develop - error only occurred while using Drupal CMS with duplicate functionality turned on, it was getting unexpected link array structure causing the error.
Fixed in https://github.com/civictheme/monorepo-drupal/pull/1357
richardgaunt → created an issue.
richardgaunt → created an issue. See original summary → .
fionamorrison23 → credited richardgaunt → .
fionamorrison23 → credited richardgaunt → .
fionamorrison23 → credited richardgaunt → .
Using the patch @robindh and it has fixed the error when using a text field as a sort field in a view.
Reviewed with the PR implemented by @alan.cole: https://github.com/civictheme/monorepo-drupal/pull/1344
Added a Merge request for easy review: https://git.drupalcode.org/project/simple_sitemap/-/merge_requests/123
Added fix to Issue fork https://git.drupalcode.org/issue/simple_sitemap-3507718
richardgaunt → created an issue.
gargsuchi → credited richardgaunt → .
We have PRs under review:
https://github.com/civictheme/uikit/pull/480
and
https://github.com/civictheme/monorepo-drupal/pull/1335
The fix has been merged.
There is another outstanding issue with 11 we are working on and will have a fix shortly.
Thank you for reporting the issue and investigating
Implemented with:
https://github.com/civictheme/uikit/pull/477
https://github.com/civictheme/monorepo-drupal/pull/1331
PR ready for review