fix: #3610651 Stop re-declaring the events listing, and match the README to the project page.

Rajab Natshahrequested to merge
issue/educare-3610651:issue/educare-3610651-events-listing-ownership into 1.0.x

Educare's views.view.events config action carried a near-clone of the display map that Varbase Events Base already ships. simpleConfigUpdate replaces the value at the key path - it does not merge - so Educare silently overrode the base recipe's entire listing: its style, its exposed filters, its pager, its row and its fields. Any change made in Varbase Events Base had no effect on an Educare site; that is how a filter removed in the base recipe kept appearing on /events.

The action is now dotted leaf paths, setting only what is genuinely Educare's: the display titles, the /events/upcoming and /events/past paths and their menu settings, and the block display's link. The listing itself is owned by Varbase Events Base, and the theme is applied by the setViewsComponentStyleTheme config action from Varbase Recipes.

Second fix in the same action: the events listing on the /events Canvas page is the block display, whose link_display was the view's own page display, so Views built the exposed form's action from that page's path - pressing Apply Filter navigated the visitor away from /events to /events/upcoming, a bare view page with no hero, title or breadcrumb. Set to link_display: custom_url / link_url: events, so the form posts back to /events and the filtered results render in place.

A rule for reviewers, worth stating: a site template must override a base recipe with leaf paths, never with a whole display map.

README.md is rewritten to match the drupal.org project page: the composer create-project drupal/varbase_project:~11.0.0 + composer require drupal/educare:1.0.x-dev install, the DDEV local setup, and the note that Educare is a type: Site recipe chosen in the installer's "Choose a site template" step rather than applied with drush recipe on an installed site (the old README said the latter, which does not work). It also records which base recipes provide the Events and News sections.

Issue

https://www.drupal.org/project/educare/issues/3610651

Related

One change split across three projects, cross-linked:

Verified together on a fresh drush site:install of the Educare site template: 0 errors, the events listing exposes Search by + Type, the grid renders through vartheme_bs5_educare:views-view-grid, and /events, /events/upcoming and /events/past all return HTTP 200. The filter row was checked in a real browser.

AI-Generated: Yes (Used Claude Code to switch the events listing config action to leaf-path overrides, fix the /events block display link, and rewrite the README to match the drupal.org project page.)

Checkpoints

  • File an issue about this project
  • Addition/Change/Update/Fix to this project
  • Testing to ensure no regression
  • Automated unit/functional testing coverage
  • Developer Documentation support on feature change/addition
  • User Guide Documentation support on feature change/addition
  • UX/UI designer responsibilities
  • Accessibility and Readability
  • Reviewed by a human
  • Code review by maintainers
  • Full testing and approval
  • Credit contributors
  • Review with the product owner
  • Update Release Notes
  • Release
Edited by Rajab Natshah

Merge request reports