Skip to content

Resolve canvasGradient is undefined in node - #10328

Merged
etimberg merged 5 commits into
chartjs:masterfrom
LeeLenaleee:patch-1
May 3, 2022
Merged

etimberg merged 5 commits into
chartjs:masterfrom
LeeLenaleee:patch-1

Conversation

@LeeLenaleee

Copy link
Copy Markdown
Collaborator

resolves #10327

@LeeLenaleee LeeLenaleee added this to the Version 3.8.0 milestone Apr 29, 2022
etimberg
etimberg previously approved these changes Apr 29, 2022
@etimberg
etimberg requested a review from kurkle May 1, 2022 15:05
etimberg
etimberg previously approved these changes May 1, 2022
kurkle
kurkle previously approved these changes May 1, 2022

@kurkle kurkle left a comment

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.

@etimberg what do you think of exporting this?
Could be used in plugins

@etimberg

etimberg commented May 1, 2022

Copy link
Copy Markdown
Member

Sure, we could add an export here

@LeeLenaleee

Copy link
Copy Markdown
Collaborator Author

Will update PR to export function and typings tomorrow

@amitkumarhq amitkumarhq left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Sure!

amitkumarhq
amitkumarhq previously approved these changes May 2, 2022
@LeeLenaleee
LeeLenaleee dismissed stale reviews from amitkumarhq, kurkle, and etimberg via 6a6270f May 2, 2022 21:08
@LeeLenaleee

Copy link
Copy Markdown
Collaborator Author

@amitkumarhq leaving a review or approval on the code once is fine but please dont spam approvals, approving 3 times within 17 minutes and then again after 14.5 hours won't help, it only iritates people as they will get notifications from it. So please consider that we also have normal lives, that this is volunteer work and we can't always implement feedback right the second we see it.

etimberg
etimberg previously approved these changes May 2, 2022
Comment thread types/helpers/helpers.color.d.ts Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ReferenceError: CanvasGradient is not defined

4 participants