Skip to content

Commit

Permalink
add pie chart axes removal
Browse files Browse the repository at this point in the history
  • Loading branch information
@alexfauquette
alexfauquette committed Nov 5, 2024
1 parent f391023 commit c41b3ae
Showing 1 changed file with 8 additions and 0 deletions.
8 changes: 8 additions & 0 deletions docs/data/migration/migration-charts-v7/migration-charts-v7.md
View file
Original file line number Diff line number Diff line change
Expand Up @@ -88,3 +88,11 @@ To pass props to the legend, use the `slotProps.legend`.
- <PieChart legend={{ ... }} />
+ <PieChart slotProps={{ legend: { ... } }} />
```

## Remove Pie Chart axes

The `<PieChart />` got by error the code to render axes.
This code is removed in v8, which implies removing the following props: `axisHighlight`, `topAxis`, `rightAxis`, `bottomAxis`, and `leftAxis`.

This should not impact your code.
If you used axes in a pie chart please open an issue, we would be curious to get more information about the use-case.

Check warning on line 98 in docs/data/migration/migration-charts-v7/migration-charts-v7.md

View workflow job for this annotation

GitHub Actions / runner / vale 

[vale] reported by reviewdog 🐶
[Google.We] Try to avoid using first-person plural like 'we'.

Raw Output:
{"message": "[Google.We] Try to avoid using first-person plural like 'we'.", "location": {"path": "docs/data/migration/migration-charts-v7/migration-charts-v7.md", "range": {"start": {"line": 98, "column": 55}}}, "severity": "WARNING"}

Check failure on line 98 in docs/data/migration/migration-charts-v7/migration-charts-v7.md

View workflow job for this annotation

GitHub Actions / runner / vale 

[vale] reported by reviewdog 🐶
[MUI.CorrectReferenceAllCases] Use 'use case' instead of 'use-case'

Raw Output:
{"message": "[MUI.CorrectReferenceAllCases] Use 'use case' instead of 'use-case'", "location": {"path": "docs/data/migration/migration-charts-v7/migration-charts-v7.md", "range": {"start": {"line": 98, "column": 109}}}, "severity": "ERROR"}

0 comments on commit c41b3ae

Please sign in to comment.