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.

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

How can I open a PR #56

Closed
brandanking-decently opened this issue Nov 25, 2024 · 4 comments
Closed

How can I open a PR #56

brandanking-decently opened this issue Nov 25, 2024 · 4 comments

Comments

@brandanking-decently
Copy link
Contributor

Subject

Pull Requests

Description

I have created a small update to separate out some of the logic so it is not all just inside the useEcharts hook. I would like to provide you with a PR but currently I am unable too. Simple changes mainly, but would have minor performance impacts as functions aren't being recreated on every render etc

@hugocxl
Copy link
Owner

hugocxl commented Nov 26, 2024

Hi @brandanking-decently, can't you open a PR? what's your issue exactly?

@brandanking-decently
Copy link
Contributor Author

Hey @hugocxl when trying to push a new branch to create a PR from I can't. The PR itself isn't one that would specifically fix anything (does fix animations not being set by default) but I've separated out the code more so that the useEcharts hook doesn't have so many functions that could be outside of it. Reducing the times the functions are initialised etc

@hugocxl
Copy link
Owner

hugocxl commented Nov 26, 2024

mmm... seems not related to the repo itself. Have you forked the project?

@brandanking-decently
Copy link
Contributor Author

Hey @hugocxl I closed this as I was able to create a PR. It would be good to get some feedback on this as it also fixes #37. At least from the testing I did myself

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

No branches or pull requests

2 participants