rails_live_reload icon indicating copy to clipboard operation
rails_live_reload copied to clipboard

config.enabled config not working

Open MichalRemis opened this issue 2 years ago • 5 comments
trafficstars

I am using multiple development environments, but I found that rails_live_reload doesn't work in environments named other than "development". I have set the config in initializer:

config.enabled = Rails.env.development? || Rails.env.development_pl? || Rails.env.development_cz?

But it doesn't work. It's probably because initializers/rails_live_reload.rb is evaluated after rails_live_reload/engine.rb is loaded, where there is condition checking this (if RailsLiveReload.enabled? && defined?(::Rails::Server)...), which therefore doesn't use the config from intializer.

Is there please a way to make live reload work in environments other than 'development'?

I am using rails (6.1.4.1) and rails_live_reload (0.3.4)

MichalRemis avatar Jun 21 '23 20:06 MichalRemis

hmm, interesting case @MichalRemis can you try to contribute to the gem?

igorkasyanchuk avatar Jun 23 '23 06:06 igorkasyanchuk

@igorkasyanchuk sorry I am too busy atm.. Maybe I would just remove config.enabled option from docs, since it doesn't work

MichalRemis avatar Jun 23 '23 08:06 MichalRemis

no, removing it from doc has no value

I would just suggest to wait when you will have time, us or someone else

this setting should be modified

https://github.com/railsjazz/rails_live_reload/blob/5b86eb7630b256cad46b7a8393ff67bf07c0a33e/lib/rails_live_reload/config.rb#L28

for now you can just fork and edit it by yousefl

igorkasyanchuk avatar Jun 23 '23 12:06 igorkasyanchuk

and if someone want's to fix, a possible approach can be like this https://github.com/igorkasyanchuk/rails_performance/blob/master/lib/rails_performance.rb#L55-L56

igorkasyanchuk avatar Jun 23 '23 12:06 igorkasyanchuk

I suppose noone had time yet? I just stumbled over the config option not working, I tried to make it configurable in our project like this:

config.enabled = (Rails.env.development? and (ENV.fetch("DISABLE_LIVE_RELOAD", nil) != "true"))

and also found that the setting seems to have no effect..

MGPalmer avatar Feb 07 '24 13:02 MGPalmer