- Issue created by @larowlan
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
We should expand our test coverage to include more field types and cardinalities
Cardinalities: π Multiple fields widget should fully work in the contexual form Active .
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
Linked 3 issues @bnjmnm created for widgets.
Can you and @bnjmnm decide how to approach breaking up this work? π
- First commit to issue fork.
- πΊπΈUnited States bnjmnm Ann Arbor, MI
I expected way more of the fields added in
xb_test_article_fields
to either "just work", or not work in a manner that didn't adversely impact the form if they weren't fields that had value changes.In the case of language, options-as-buttons (radios), and possibly others, it's clear this isn't the case - they are enough to deserve their own issues IMO. In particular, if the problems impact the ability to test any fields in the form, a separate issue might be good. This issue could then be specific to adding tests for fields that either work already, or are broken in a way that doesn't contaminate the entire form.
It looks like the branch here and the MR in π Get Options as buttons in Page Data form working Active are tackling the options-as-buttons issue in different places and I suspect they might be more complementary than conflicting. My preference is to see the work on option buttons here brought over to that issue. BTW I describe a wall I hit in comment 4 β but perhaps the approach here gets around that.
- π¦πΊAustralia larowlan π¦πΊπ.au GMT+10
I'll hoist my work into your branch ποΈ
- π¦πΊAustralia larowlan π¦πΊπ.au GMT+10
Merged forces with π Get Options as buttons in Page Data form working Active