Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SearchKit - Fix selection of fields when creating a Data Segment #23265

Merged
merged 1 commit into from
Apr 21, 2022

Conversation

colemanw
Copy link
Member

@colemanw colemanw commented Apr 20, 2022

Overview

Fixes bug documented in #23059 (comment)

Before

Creating a SearchSegment in the UI doesn't work right for fields with options.

After

Works.

Note

Any broken segments created while testing the last PR should be deleted and recreated with the fixed UI.

@civibot
Copy link

civibot bot commented Apr 20, 2022

(Standard links)

@eileenmcnaughton
Copy link
Contributor

Just did an r-run & this nails it

@eileenmcnaughton
Copy link
Contributor

test this please

@eileenmcnaughton
Copy link
Contributor

Test fails somehow relate to previous merge but not this one

Test Result (3 failures / ±0)api\v4\SearchDisplay\SearchSegmentTest.testRangeSearchSegmentapi\v4\SearchDisplay\SearchSegmentTest.testSegmentCustomFieldapi\v4\SearchDisplay\SearchSegmentTest.testSegmentCalcField

Test Result (3 failures / ±0)
api\v4\SearchDisplay\SearchSegmentTest.testRangeSearchSegment
api\v4\SearchDisplay\SearchSegmentTest.testSegmentCustomField
api\v4\SearchDisplay\SearchSegmentTest.testSegmentCalcField

@eileenmcnaughton eileenmcnaughton merged commit b142a92 into civicrm:master Apr 21, 2022
@eileenmcnaughton eileenmcnaughton deleted the pivotTablesFix branch April 21, 2022 04:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants