Skip to content

Conversation

@bcb37
Copy link
Collaborator

@bcb37 bcb37 commented Apr 9, 2025

No description provided.

@bcb37 bcb37 linked an issue Apr 9, 2025 that may be closed by this pull request
@bcb37 bcb37 requested review from danoswaltCL and zackcl April 9, 2025 21:10
@zackcl
Copy link
Collaborator

zackcl commented Apr 10, 2025

@bcb37 I'm working on the Segment Import List modal (#2335) and was wondering if we also need '/segments/lists/import/validation' endpoint like Feature Flag:

validateFeatureFlagList: '/flags/lists/import/validation',
importFeatureFlag: '/flags/import',
importFeatureFlagList: '/flags/lists/import',

@bcb37
Copy link
Collaborator Author

bcb37 commented Apr 10, 2025

@bcb37 I'm working on the Segment Import List modal (#2335) and was wondering if we also need '/segments/lists/import/validation' endpoint like Feature Flag:

validateFeatureFlagList: '/flags/lists/import/validation',
importFeatureFlag: '/flags/import',
importFeatureFlagList: '/flags/lists/import',

We can just use the segment validation endpoint for that. No need for a separate one, I don't think.

@danoswaltCL
Copy link
Collaborator

@zackcl are we good to merge this one?

@zackcl zackcl merged commit c3a0961 into dev Apr 11, 2025
8 checks passed
@zackcl zackcl deleted the feature/2339-import-list-endpoint branch April 11, 2025 10:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Import List Endpoint

4 participants