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

Better client side support #107

Closed
wants to merge 2 commits into from
Closed

Better client side support #107

wants to merge 2 commits into from

Conversation

sfigueiras
Copy link

The PR's intent is to improve the experience of using the library on the client side, the changes that were made are the following:

  • Uploaded the repository to the bower, so now it can be installed via bower install pptxgen
  • Added gulp task to generate a bundle with all the core files needed to run pptxgen (jszip, jquery)

If the PR is merged, the version tag should be bumped and the new version should be uploaded to bower, given there is no contribution instructions I did my best to be us tidy as possible. I don't now how you handle builds internally, or if you do it at all so I tried not to go too far with that.
I'm using pptxgen at work and this improvement would make it easier for me to use it.

Hope it helps!

@gitbrent gitbrent self-assigned this Jun 22, 2017
gitbrent pushed a commit that referenced this pull request Jun 22, 2017
gitbrent pushed a commit that referenced this pull request Jun 22, 2017
Copy link
Owner

@gitbrent gitbrent left a comment

Choose a reason for hiding this comment

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

Great job!

@gitbrent gitbrent added this to the v1.6.0 milestone Jun 22, 2017
gitbrent pushed a commit that referenced this pull request Jun 22, 2017
@gitbrent
Copy link
Owner

Hi @santi-git ,

Thanks for the PR and doing to bower registration! I've incorporated the changes into the project.

Also, thanks for getting me started with gulp - i've been meaning to setup a workflow for a bundled file for a while now. :-)

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