rrit
Février 7, 2022, 8:33
14
Le Feature flag est implémenté.
Je ne suis pas du tout un développeur Ruby - là-dessus, j’ai vraiment besoin d’aide.
Peut-être pousser mon POC dans une nouvelle branche du dépôt discourse/discourse, avant de faire un PR sur main ?
Voici mon PR sur cette fonctionnalité :
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 Pouvez-vous me prêter votre tête pour obtenir de l’aide sur le développement de tests Rspec pour ces changements :
app/helpers/application_helper.rb : spec/helpers/application_helper_spec.rb
Je ne vois pas de tests unitaires réalisables ici. Cela semble testable uniquement par des tests d’intégration.app/models/theme.rb app/models/theme_field.rb
J’ai dû désactiver le defer tag pour le QUnit Test Runner : app/views/qunit/index.html.erb "javascript defer" = false. Et maintenant, les tests s’exécutent aussi avec "javascript defer" = true.
2 « J'aime »