Skip to content

[fix doc]-Correct spelling errors in words#420

Closed
gitforhlp wants to merge 9 commits into
apache:masterfrom
gitforhlp:master
Closed

[fix doc]-Correct spelling errors in words#420
gitforhlp wants to merge 9 commits into
apache:masterfrom
gitforhlp:master

Conversation

@gitforhlp
Copy link
Copy Markdown
Contributor

No description provided.

Comment thread en/api/action.md

```ts
// If highlight series:
// If highlight series(in the `sunburst` series,`highlight` has been deprecated since `v5.0.0`, please use `sunburstHighlight` instead):
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think sunburstHighlight is deprecated rather than highlight and your current change conflicts with your previous change in apache/echarts#17587.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Also, the changes don't match your PR title. I didn't see any fix for spelling errors.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I have modified it a moments ago,please view again, thanks

@plainheart
Copy link
Copy Markdown
Member

Superseded by #419

@plainheart plainheart closed this Aug 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants