- ๐บ๐ธUnited States smustgrave
This was previously tagged for tests in #24 which still need to happen.
Did not test for issue.
- Issue was unassigned.
- ๐ซ๐ฎFinland sokru
Related issue has tests and upgrade path that this is missing. On Views admin UI the patches from this issue uses the same label "Operator label" as โจ [PP-1] Make views exposed filter operator labels configurable Needs work , either of them would need a change the label to avoid confusion.
- First commit to issue fork.
- ๐ฎ๐ณIndia arunkumark Coimbatore
Created MR from the patch #34 and created test case for the new field.
- ๐ซ๐ฎFinland sokru
@arunmark thanks for the merge request, however this still needs the upgrade path and the config updates, similar to what has been done on โจ [PP-1] Make views exposed filter operator labels configurable Needs work .
- ๐ฎ๐ณIndia arunkumark Coimbatore
@sokru
Thanks for the comment as suggested, I added Upgrade path for the patch. I was moving to Review. - ๐ฎ๐ณIndia arunkumark Coimbatore
I verified the patch on the unit test on my local, It was working fine. Attached screenshot for reference.
Not sure why it's failing on the Automated test in Gitlab.
For now moving to review, If needed please move back to work.
- ๐บ๐ธUnited States smustgrave
Your local probably isnโt 100% aligned with gitlab. Canโt be marked with a failing test