rrit
7 Febbraio 2022, 8:33pm
14
Il feature flag è implementato.
Non sono affatto uno sviluppatore Ruby, su questo ho decisamente bisogno di aiuto.
Forse dovrei caricare il mio POC in un nuovo branch nel repository discourse/discourse, prima di fare un PR su main?
Questo è il mio PR su questa funzionalità:
main ← rr-it:dev/javascript-defer
opened 03:49AM - 08 Feb 22 UTC
Implement experimental feature flags for "static topic content" and " javascript… defer"
### static topic content
`SiteSetting.enable_experimental_static_topic_content`
Can be set via settings dashboard.
Show static content in topic view for a faster Largest Contentful Paint (LCP).
Warning: visitors might see disconcerting jumping of content!"
### javascript defer
`SiteSetting.enable_experimental_javascript_defer`
Can be set via settings dashboard.
Defer loading of JavaScripts to show static content even faster in combination with 'enable_experimental_javascript_defer'. Warning: the discourse frontend, themes, components and/or plugins might fail!
---
The vendor-javascript and all preceding javascripts are not deferred right now.
@see: https://github.com/rr-it/discourse/commit/49405c353a31180933aed9dca7697dce05227707
Ideas on how to solve this are very welcome.
---
For more information see also:
https://meta.discourse.org/t/defer-javascript-and-show-interim-content-on-initial-page-load/216458
@david Puoi darmi una mano nello sviluppo di test Rspec per queste modifiche:
app/helpers/application_helper.rb : spec/helpers/application_helper_spec.rb
Non vedo test unitari fattibili qui. Sembra testabile solo tramite test di integrazione.app/models/theme.rb app/models/theme_field.rb
Ho dovuto disabilitare il defer tag per QUnit Test Runner: app/views/qunit/index.html.erb "javascript defer" = false. E ora i test vengono eseguiti anche con "javascript defer" = true.
2 Mi Piace