- Issue created by @ducktape
- 🇧🇪Belgium ducktape
Added the fix as a MR for 3.0.x.
Added a patch for 2.x for who might need it in their workflow.
- Status changed to Needs review
4 months ago 4:17pm 20 August 2024 - 🇮🇱Israel jsacksick
hm... Then we have the same issue with other views? We have 10 occurrences of plugin_id: numeric in the current codebase?
- Status changed to Needs work
3 months ago 1:36pm 22 August 2024 - 🇳🇴Norway zaporylie
We need a update_hook_N similar to what was done in https://git.drupalcode.org/project/commerce/-/merge_requests/300/diffs#8... to update views on existing commerce instances if they were not customized by site admin.
- 🇮🇱Israel jsacksick
As discussed on Slack, let's check how core handled this, and let's open a followup issue for the update hook.
-
jsacksick →
committed 41ea9def on 3.0.x
Issue #3469240 by ducktape, keshavv, jsacksick, zaporylie: The update to...
-
jsacksick →
committed 41ea9def on 3.0.x
- 🇮🇱Israel jsacksick
Committed the patch so the views are at least correct on news installs, but we still need to take care of the update hook.