Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
In #1810480: Provide the plugin_id to support views metadata integration dawehner painstakingly added the plugin_id to each handler in each export. But in #1851086: Replace admin/people with a View, I tried to add a new view and didn't notice it was missing them.
We don't necessarily care that this is checked on runtime, but we should know during tests.
Eventually we could consider adding it to something like #1842956: [Meta] Implement event listeners to validate imports, but in the meantime we could test each view as it passes through ViewTestData::importTestViews().
Comment | File | Size | Author |
---|---|---|---|
#13 | 1915686-13.patch | 505 bytes | Lendude |
Comments
Comment #13
LendudeI think we catch this now, lets see if this fails when I remove a plugin_id
Comment #14
LendudeNope, we don't catch that :(
Comment #15
LendudePostponing this on #3364108: Configuration schema & required keys and maybe some of the child issues this will spawn
Comment #16
Lendude