Skip to content
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.

Sign up for GitHub

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

Jump to bottom

fix(web): fix line chart using interval field as xAxis & fix pie chart tooltip #1697

Merged
merged 1 commit into from
Jul 17, 2024
Merged

fix(web): fix line chart using interval field as xAxis & fix pie chart tooltip #1697

merged 1 commit into from
Jul 17, 2024

Conversation

chenluli
Copy link
Contributor

@chenluli chenluli commented Jul 5, 2024

  1. Previously, the line chart required the x-axis field to be of string type, which caused the chart to fail in cases where there was no string type field. Now, if there is no string type field, the first field will be used as the x-axis.
Screenshot 2024-07-05 at 13 50 03

2.add tooltip title for pie chart

Screenshot 2024-07-05 at 13 50 12

@github-actions github-actions bot added fix Bug fixes web Module: web labels Jul 5, 2024
@Aries-ckt
Copy link
Collaborator

/assign @Aralhi

csunny
csunny approved these changes Jul 16, 2024
View reviewed changes
Copy link
Collaborator

@csunny csunny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

r+

2976151305
2976151305 approved these changes Jul 17, 2024
View reviewed changes
Copy link
Collaborator

@2976151305 2976151305 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

r+

@csunny csunny merged commit 083becd into eosphoros-ai:main Jul 17, 2024
3 checks passed
Hopshine pushed a commit to Hopshine/DB-GPT that referenced this pull request Sep 10, 2024
@chenluli @Hopshine
fix(web): fix line chart using interval field as xAxis & fix pie char…
8caaeaf 
…t tooltip (eosphoros-ai#1697)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@2976151305 2976151305 2976151305 approved these changes

@csunny csunny csunny approved these changes

Assignees
No one assigned
Labels
fix Bug fixes web Module: web
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@chenluli @Aries-ckt @csunny @2976151305