- Issue created by @penyaskito
- Merge request !10530Issue #3493296 by penyaskito: Views exposed sort identifiers are not configurable when also exposed β (Open) created by penyaskito
- πͺπΈSpain penyaskito Seville π, Spain πͺπΈ, UTC+2 πͺπΊ
This:
- needs tests
- need upgrade path tests.
- needs updating provided views in all profiles (and recipes?).Otherwise might be ready. Marking Needs review for getting feedback before working in tests.
- π³π±Netherlands Lendude Amsterdam
I think the feature makes sense and is probably much harder to do in contrib than adding it to core since this stuff isn't exposed in any sort of hook.
I'm not sold on the names and descriptions used here. I'd use 'key' and not 'id' to describe these, but even than it's still pretty hard to grasp what these are for. Since this is some pretty specialised config, we need strong descriptions to make clear what these are for I feel.
- πͺπΈSpain penyaskito Seville π, Spain πͺπΈ, UTC+2 πͺπΊ
@lendude Given this manifested as an error message, I was tempted to mark this even as a bug instead of Feature request.
Naming these are hard. In the meantime I used suffix as you suggested.
The descriptions definitely need work. - πͺπΈSpain penyaskito Seville π, Spain πͺπΈ, UTC+2 πͺπΊ
This now contains tests, including for the views upgrade.