Request: make `activate_js` optional within `bundle_resources`

[LDSamson]

Related to #503.

The use of activate_js is hard coded here:

golem/R/bundle_resources.R

Line 58 in 6291796

golem::activate_js()

This will inject javascript from the golem-js.js file in the header with multiple custom message handlers:

golem/R/js.R

Line 43 in 6291796

system.file("utils/golem-js.js", package = "golem")

While you care considering deprecating the functions, according to #503, would you in the meantime consider making it possible to disable adding these custom message handlers within the function bundle_resources?

Our reason for getting rid of these functions are: we are currently not using them, and it provides an extra hurdle to create a strict and secure content-security-policy that we can use with our production-ready applications.

[ColinFay]

That does make sense, and should be easy to implement without any breaking change on the current behavior.

Will embark this in the next release.

[ColinFay]

Implemented in #1195

[LDSamson]

Thank you for the follow-up and the implementation!

On a side-note: I love the sparkles option in this function, it's a hidden 💎 😄