kgabryje commented on code in PR #37639:
URL: https://github.com/apache/superset/pull/37639#discussion_r2759946937
##########
superset/mcp_service/chart/schemas.py:
##########
@@ -692,7 +696,11 @@ class XYChartConfig(BaseModel):
False,
description="Stack bars/areas on top of each other instead of
side-by-side",
)
- group_by: ColumnRef | None = Field(None, description="Column to group by")
+ group_by: ColumnRef | None = Field(
+ None,
+ description="Column to group by (creates series/breakdown). "
+ "Use this field for series grouping — do NOT use 'series'.",
Review Comment:
I think that's good enough for now since we support only a few chart types,
but FYI in case we add support for more in the future - some viz plugins (e.g.
bubble, word cloud) DO have a 'series' field in their form data
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
