-
Notifications
You must be signed in to change notification settings - Fork 715
feat: dashboard table chart show data as json #9327
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
Conversation
|
Failed to generate code suggestions for PR |
4650057 to
e9f5a77
Compare
|
Failed to generate code suggestions for PR |
e9f5a77 to
a3eae0f
Compare
Greptile OverviewGreptile SummaryThis PR adds a new feature to dashboard table charts that allows users to render specific fields as formatted JSON/Array data. Key Changes:
The implementation follows existing patterns in the codebase (similar to Confidence Score: 5/5
Important Files ChangedFile Analysis
Sequence DiagramsequenceDiagram
participant User
participant QueryBuilder as DashboardQueryBuilder
participant Panel as useDashboardPanel
participant Converter as convertTableData
participant Renderer as TableRenderer
participant JSON as JsonFieldRenderer
User->>QueryBuilder: Toggle "Render Data as JSON" checkbox
QueryBuilder->>Panel: Update field.showFieldAsJson = true
Panel->>Converter: Pass panelSchema with showFieldAsJson
Converter->>Converter: Map showFieldAsJson to column object
Converter->>Renderer: Return columns with showFieldAsJson flag
Renderer->>Renderer: Check props.col.showFieldAsJson
alt showFieldAsJson is true
Renderer->>JSON: Render with JsonFieldRenderer
JSON->>JSON: Parse and format JSON/Array
JSON-->>Renderer: Return formatted output
else showFieldAsJson is false
Renderer->>Renderer: Render plain value
end
Renderer-->>User: Display table cell
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
6 files reviewed, no comments
a3eae0f to
d6870e2
Compare
…hboardQueryBuilder
d6870e2 to
e9db659
Compare
No description provided.