Added route information for groups/groupby params#127
Added route information for groups/groupby params#127akhilmmenon wants to merge 2 commits intomasterfrom
Conversation
|
@akhilmmenon have we verified it on swagger validator? |
| filtering by metadata, time range, and other standard analytics filters. Use the `columns` parameter | ||
| to specify which metrics to include in the response. | ||
| parameters: | ||
| - name: groupBy |
There was a problem hiding this comment.
you will need to add query parameters here as well like this (replacew with the query params for this route)
- name: current_page
in: query
schema:
type: integer
example: "0"
- name: name
in: query
schema:
type: string
example: "workspace"
description: "Workspace name to filter results, case sensitive"
can you remove them from the bottom, those are shared query params for all the analytics routes, I think these query params are only for this route, please check the code I might be wrong
|
@narengogi I checked the code, and other than columns and include_total params, all the other params are shared across various endpoints, only those two are specific to groupby endpoint. So I have removed it from components/parameters and made them inline in the groupby route. @siddharthsambharia-portkey I have validated the openAPI spec in Swagger Editor online and also validated it in the CLI |

Added the route information for /analytics/groups/{groupBy} which returns analytics data grouped by the specified dimension. Supports configurable metric columns, filtering by metadata, time range, and other standard analytics filters. Use the columns parameter to specify which metrics to include in the response.