DEV: Move browser-update.js.erb out of vendored scripts (PR #11234)

Follow up to @eviltrout’s comment.

GitHub

Thanks for following up. We have another problem now though - .erb doesn’t work in the ember cli world. Would it be possible to update this to be pure JS? If not we use the update_constants rake task. That might not work in this case due to a translation.

This pull request introduces 1 alert when merging 2b215fea5ea0780927145bfabf67497984f5dc5e into 8e8dca9246d1762c2e9e113471bd21b8e41b5563 - view on LGTM.com

new alerts:

  • 1 for Superfluous trailing arguments

Thanks for following up. We have another problem now though - .erb doesn’t work in the ember cli world. Would it be possible to update this to be pure JS? If not we use the update_constants rake task. That might not work in this case due to a translation.

@eviltrout - you might want to take a look at https://github.com/discourse/discourse/pull/11267 - it should solve this part of the problem.