feat: add sort and filters to project-insights#3922
Conversation
Signed-off-by: Gašper Grom <gasper.grom@gmail.com>
|
|
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
2 similar comments
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
There was a problem hiding this comment.
Pull request overview
Adds sorting and filtering capabilities to the project_insights Tinybird pipe, allowing consumers to filter by LFX status and specify sort field/direction.
Changes:
- Added
isLfxfilter parameter to restrict results to LFX or non-LFX projects - Added
orderByFieldandorderByDirectionparameters for sorting, withname ASCas secondary sort - Updated pipe description with new parameter documentation
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| String( | ||
| orderByField, "name", description="Order by field.", required=False |
| {% if String( | ||
| orderByDirection, | ||
| 'asc', | ||
| description="Order by direction. ASC or DESC", | ||
| required=False, | ||
| ) == 'asc' or String( | ||
| orderByDirection, | ||
| 'asc', | ||
| description="Order by direction. ASC or DESC", | ||
| required=False, | ||
| ) == 'ASC' %} ASC |
Note
Medium Risk
Adds client-controlled filtering and dynamic
ORDER BYto theproject_insightsquery, which can change result sets and query performance and needs careful validation of allowed sort columns/directions.Overview
Adds new optional query parameters to
project_insights.pipe: anisLfxfilter (mapped toisLF) and dynamic sorting viaorderByField/orderByDirection(defaulting toname/ascending), with a deterministic secondary sort byname.Written by Cursor Bugbot for commit d406322. This will update automatically on new commits. Configure here.