Skip to content

Conversation

@zackcl
Copy link
Collaborator

@zackcl zackcl commented Sep 9, 2024

Closes #1775

Resolves the comment: #1775 (comment)

@zackcl zackcl marked this pull request as ready for review September 9, 2024 14:18
@zackcl
Copy link
Collaborator Author

zackcl commented Sep 9, 2024

@bcb37 @VivekFitkariwala @Yagnik56 I can confirm that simply updating segment: segment.id to segment: segment for querying the users to delete fixes the issue. The values in the Include/Exclude list are properly updated, even when values are deleted. Please feel free to approve this PR.

@zackcl zackcl merged commit 25496d5 into dev Sep 10, 2024
@zackcl zackcl deleted the bugfix/1775-remove-list-values branch September 10, 2024 08:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

FF: Implement the Edit Button for the "Include/Exclude Lists" Table Rows

4 participants