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

Replace jquery each by standard forEach #418

Merged
merged 1 commit into from
Oct 13, 2018

Conversation

fdussert
Copy link
Contributor

I don't know why but this $.each don't work in a node/react app.
It triggers an error and prevent the file to be generated and saved

I don't know why but this $.each don't work in a node/react app.
It triggers an error and prevent the file to be generated and saved
@gitbrent gitbrent merged commit a01f127 into gitbrent:master Oct 13, 2018
@gitbrent gitbrent self-assigned this Oct 13, 2018
@gitbrent gitbrent added this to the 2.4.0 milestone Oct 13, 2018
@gitbrent
Copy link
Owner

@fdussert - thanks for reporting this and creating a pull request.

gitbrent added a commit that referenced this pull request Oct 13, 2018
@cinnamonKale
Copy link

@gitbrent any idea when this is scheduled to be published to npm?

@gitbrent
Copy link
Owner

Probably by month's end. SVG support is still needed for non-browsers, otherwise, 2.4.0 is ready.

ericrockey pushed a commit to WeTransfer/PptxGenJS that referenced this pull request Mar 24, 2020
ericrockey pushed a commit to WeTransfer/PptxGenJS that referenced this pull request Mar 24, 2020
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.

3 participants