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

Jump to bottom

fix typos, clarify #15512

Merged

pissang merged 2 commits into apache:master from fredsterorg:master

Aug 13, 2021

Contributor

fredster33 commented Aug 11, 2021

Brief Information

This pull request is in the type of:

[] bug fixing
new feature
others

What does this PR do?

Fix typos and make the docs clearer.

Fixed issues

Typos

Details

Before: What was the problem?

There were some typos.

After: How is it fixed in this PR?

There are no typos.

Misc

The API has been changed (apache/echarts-doc#xxx).
This PR depends on ZRender changes (ecomfe/zrender#xxx).

Related test cases or examples to use the new APIs

NA.

Others

Merging options

Please squash the commits into a single one when merge.

Other information

fredster33 added 2 commits


          Fix typos

1859c8c


          fix typos, clarify

5d09b3b

echarts-bot bot commented Aug 11, 2021

Thanks for your contribution!
The community will review it ASAP. In the meanwhile, please checkout the coding standard and Wiki about How to make a pull request.

pull-request-size bot added the size/S label

echarts-bot bot added PR: awaiting review PR: first-time contributor labels

plainheart approved these changes

View changes

pissang merged commit 0d5496f into apache:master

1 check passed

echarts-bot bot removed the PR: awaiting review label

echarts-bot bot commented Aug 13, 2021

Congratulations! Your PR has been merged. Thanks for your contribution! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment