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

Add handlebars templating #3

Merged
merged 2 commits into from
Aug 29, 2022
Merged

Add handlebars templating #3

merged 2 commits into from
Aug 29, 2022

Conversation

drjonnicholson
Copy link
Contributor

Hey, starting to look at Bookmarklets for something and found your plugin. Thought I'd have a go at adding the template support mentioned. This PR:

  • Adds Handlebars template engine
  • Migrated HTML to hbs template as default
  • Added ability to pass a path to a template file
  • Added examples for custom HTML and custom React component output

Let me know what you think

@tomrule007
Copy link
Owner

@drjonnicholson thanks for taking the time to create this pr! Give me a few days to review it and hopefully we can get it merged in soon :)

@drjonnicholson
Copy link
Contributor Author

No rush @tomrule007 - any changes let me know, happy to update

@drjonnicholson
Copy link
Contributor Author

Hey @tomrule007, any thoughts / feedback?

@tomrule007 tomrule007 merged commit a641d13 into tomrule007:master Aug 29, 2022
@tomrule007
Copy link
Owner

@drjonnicholson thanks for the contribution!

I updated the Readme giving you credit for the template engine change and also published the new version.

https://www.npmjs.com/package/bookmarklet-webpack-plugin/v/1.1.0

@drjonnicholson drjonnicholson deleted the add-handlebars-templating branch July 19, 2023 06:57
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

Successfully merging this pull request may close these issues.

None yet

2 participants