Loading
fix: #3622247 Fix the RightUp site template install and add functional CI
Issue: https://www.drupal.org/project/rightup/issues/3622247
What changed
- Removed 21 broken Canvas component config files (
config/canvas.component.sdc.vartheme_bs5_rightup.*.ymlwithactive_version: fallbackand an emptylast_active_version) and their 21 matchingdisable: []actions inrecipe.yml. Both halves have to go together — deleting only the files fails the install withEntity ... does not exist. - RightUp now ships its own media: 18 media entities, 17 file entities and 17 binaries under
content/, keeping the existing UUIDs so the Canvas pages resolve unchanged. The recipe is self-contained — novarbase_media_assetsorvarbase_demo_contentdependency. canvas_page_template_componentadded toinstall:, so Canvas does not install it duringRecipeAppliedEventand rebuild the container mid-dispatch.- CI replaced with the Drupal CMS pipeline — install job
🧩 Drupal CMS - RightUp, plus thefunctionalandfunctional (smoke)jobs running the varbase-e2e suite undertests/. The content guard checks Canvas pages only, since RightUp ships no demo nodes by design (#3621686).
Also renames Rightup to RightUp in the three scripts/ files.
Verification
Run locally today with no patches and nothing pinned — Drupal core 11.4.6, PHP 8.4, canvas 1.11.0:
| Base | Before | After |
|---|---|---|
| Drupal CMS | exit 1, 379 warnings, 1 error | exit 0, 0 warnings, 0 errors |
| Varbase project | — | exit 0, 0 warnings, 0 errors |
On both, after install: default theme vartheme_bs5_rightup, 18 media, 5 Canvas pages, and /, /features, /blog, /about-varbase, /contact-us, /user/login all HTTP 200. The Drupal CMS browser installer path was also driven through Playwright and produces the same working site.
AI-Generated: Yes
Checkpoints:
- File an issue
- Addition/Change/Update/Fix
- Testing to ensure no regression
- Automated unit testing coverage
- Automated functional testing coverage
- UX/UI designer responsibilities
- Readability
- Accessibility
- Performance
- Security
- Developer Documentation
- User Guide Documentation
- Reviewed by human
- Code review by maintainers
- Full testing and approval
- Credit contributors
- Review with the product owner
- Release notes snippet
- Release
Edited by Rajab Natshah