kernelci / kernelci/kernelci-api

Investigate `fastapi-pagination` package for getting response without alias

Open
#276 0 comments 0 reactions 1 assignee Claimed by @JenySadadia View on GitHub
Dominant language
Python
Stars
10
Forks
21
PR merge metrics
No merged PRs in 30d

Description

In case we use pagination for getting objects (e.g. nodes or regressions) from API, we need to manually serialize the response model to not have alias at the moment.
Please see the detailed issue here: https://github.com/kernelci/kernelci-api/issues/246

FastAPI provides `response_model_by_alias` flag and we can set it to `False` in the endpoint handler to fix this issue.
However, this won't work in the case of `paginated` response.
So, We have fixed this issue in API. PR for the reference: https://github.com/kernelci/kernelci-api/pull/254

Investigate if we can fix the `fastapi-pagination` package for it to eliminate the requirement to serialize the response manually.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.