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

Added BAMO #348

Merged
merged 8 commits into from
Oct 3, 2024
Merged

Added BAMO #348

merged 8 commits into from
Oct 3, 2024

Conversation

tmudway
Copy link
Contributor

@tmudway tmudway commented Mar 14, 2023

No description provided.

@JannisX11
Copy link
Owner

I reviewed your submission. Two notes:

  • Blockbench already comes with JSZip by default, so you should treat it as an external dependency.
  • Please include the files that are necessary to reproduce the build, such as webpack config and package.json, in the source folder.

@tmudway
Copy link
Contributor Author

tmudway commented Apr 12, 2023

Both changes made. Anything else that looks off?

@JannisX11
Copy link
Owner

I noticed that you can still make the plugin 2x smaller if you remove the babel-loader rule from webpack config. Also please make sure to update the plugin file in the ./plugins folder.

@SirJain0
Copy link
Contributor

@tmudway Any news on how this is going? Just curious since this seems like a cool plugin

@tmudway
Copy link
Contributor Author

tmudway commented Jun 19, 2023

Updated with requested changes

@JannisX11
Copy link
Owner

Sorry this has been in the queue for so long!

@JannisX11
Copy link
Owner

Not blocking, but I would recommend including the link to documentation in a clickable way. To do that, you can use the "website" metadata field and you can additionally provide an about text.

@JannisX11 JannisX11 merged commit 10f135d into JannisX11:master Oct 3, 2024
1 check passed
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.

3 participants