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

Make the list of fields in a response configurable #36

Open
devraj opened this issue Apr 28, 2024 · 0 comments
Open

Make the list of fields in a response configurable #36

devraj opened this issue Apr 28, 2024 · 0 comments
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@devraj
Copy link
Member

devraj commented Apr 28, 2024

Is your feature request related to a problem? Please describe.
Many of the endpoints allow providing a list of fields to the request, which varies the size of the response payload (see alarms). We should support this as part of the API calls.

Describe the solution you'd like
At the moment the pyndatic models expect all of the fields to exists, we will have to study how to temporarily relax the validation requirements of pydantic models. I would rather not have to duplicate the models for the two use cases.

Provide a pythonic manner to provide the list of fields (rather than strings), again we might have to lean into the pydantic models and how they make the meta data available to the application.

Describe alternatives you've considered
NA

Additional context
NA

@devraj devraj self-assigned this Apr 28, 2024
@devraj devraj added the enhancement New feature or request label Apr 28, 2024
@devraj devraj added this to the rc-1 milestone Apr 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant