Problem/Motivation
Making the home page Stories section dynamic in #3616545 added two Views block displays to this recipe's recipe.yml: a featured display for views.view.news and a latest display for views.view.events, about three hundred and eighty lines in total.
Neither belongs here. This is a site template; the news view is owned by Varbase News Base and the events view is reshaped by Varbase Events Base. Defining the displays in the site template means no other consumer of those recipes can use them, and the recipe carries several hundred lines of view configuration it should never have owned. The exported displays also carried vartheme_bs5_educare: component ids, which only work on this template's theme.
Proposed resolution
Move both displays to the recipes that own their views, and remove them from here:
views.view.newsfeatured→ Varbase News Base (#3616585);views.view.eventslatest→ Varbase Events Base (#3616587).
Both ship with the base theme's component ids, and this recipe's existing setViewsComponentStyleTheme actions repoint them to vartheme_bs5_educare at install, exactly as they already do for every other display. The Home page keeps the two Views blocks it places; only the display definitions move.
This issue depends on both of the above being merged and released first.
Remaining tasks
- ✅ 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 educare-1.0.0-alpha2
User interface changes
- None. The Stories section renders exactly as before.
API changes
- N/A
Data model changes
- The two block displays now arrive from Varbase News Base and Varbase Events Base rather than from this recipe.
Release notes snippet
- The Stories block displays now come from Varbase News Base and Varbase Events Base, so they can be reused outside this site template.
Issue fork educare-3616588
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #6
rajab natshah✅ Released educare-1.0.0-alpha2
Comment #7
rajab natshahComment #8
rajab natshah