ace-rails-ap icon indicating copy to clipboard operation
ace-rails-ap copied to clipboard

Published 20 hours ago • verified publisher icon codykrieger

Fix snippets 404 error in production

Open benpolinsky opened this issue 8 years ago • 1 comments

This PR follows #16 to allow snippets to be accessed in production.

benpolinsky avatar Jun 25 '16 16:06 benpolinsky

@benpolinsky Sorry for the delay on this! Can you rebase on top of master to resolve the merge conflict?

My only hesitation with accepting this is that users are still required to manually include snippets in their application.js file if they want to use non-HTML, CSS, or Liquid templates. Those defaults might be fine for your use case, but I don't feel great about making those the defaults for everyone (especially not without documenting as such in the readme).

Thoughts? How bad is the delta in precompilation time (and asset size) if all snippets are included?

codykrieger avatar Jul 21 '16 17:07 codykrieger