Feature flag is implemented.
I’m not at all a ruby developer - on this I definitely need some help.
Maybe push my POC into a new branch in the discourse/discourse repo, before doing a PR on main?
This is my PR on this feature:
https://github.com/discourse/discourse/pull/15858
@david Can you lend me your head for some help on developing Rspec tests for these changes:
app/helpers/application_helper.rb: spec/helpers/application_helper_spec.rb
I don’t see feasible unit tests here. It looks testable by integration tests only.
app/models/theme.rb
app/models/theme_field.rb
I had to disable defer tag for QUnit Test Runner: app/views/qunit/index.html.erb
Before QUnit Tests did still run with the feature flag "javascript defer" = false. And now the tests run also with "javascript defer" = true.