Skip to content

Commit

Permalink
Merge pull request #5843 from Countly/cohort-segment
Browse files Browse the repository at this point in the history
[star-rating] Add ignore existing cohort prop to drawer
  • Loading branch information
ArtursKadikis authored Dec 6, 2024
2 parents 547ddc2 + 6df4fde commit 998ee91
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
3 changes: 2 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,8 @@ Features:
- [dbviewer] Preventing aggregation of using any stages which might open user to harmful actions (like $merge, $out, $lookup, $uninonWith) for all users except global admin
- [nps] Fixing issues with default logo selection
- [populator] Adding ability to select features to populate and other small improvements
- [surveys] Removed unnecessary limitation with using cohorts for targeting
- [star-rating] Removed unnecessary limitation with using cohorts for targeting
- [surveys] Removed unnecessary limitation with using cohorts for targeting

Enterprise Features:
- [cohorts] Adding ability to edit cohorts. This deletes historical calculations
Expand Down
6 changes: 4 additions & 2 deletions plugins/star-rating/frontend/public/templates/drawer.html
Original file line number Diff line number Diff line change
Expand Up @@ -373,7 +373,9 @@ <h3 class="dropzone-custom-title">
:show-in-the-last-hours="true"
ref="ratingsSegmentation"
:property-segmentation="drawerScope.editedObject.targeting.user_segmentation"
:behavior-segmentation="drawerScope.editedObject.targeting.steps">
:behavior-segmentation="drawerScope.editedObject.targeting.steps"
:ignore-existing-cohort="true"
>
</cly-cohort-segmentation>
</div>
</div>
Expand Down Expand Up @@ -412,4 +414,4 @@ <h3 class="dropzone-custom-title">
</cly-form-step>
</template>
</cly-drawer>
</div>
</div>

0 comments on commit 998ee91

Please sign in to comment.