Merge pull request #13165 from chahmedejaz/bugfix/unable-to-view-images-in-dev-env

Fix base urls for assets in development

.env.development

@@ -18,7 +18,7 @@ OFN_REDIS_URL="redis://localhost:6379/1"
 OFN_REDIS_JOBS_URL="redis://localhost:6379/2"
 OFN_REDIS_CABLE_URL="redis://localhost:6379/0"
 
-SITE_URL="0.0.0.0:3000"
+SITE_URL="localhost:3000"
 
 # Deactivate rack-timeout in development.
 # https://github.com/zombocom/rack-timeout#configuring

app/mailers/application_mailer.rb

@@ -18,6 +18,7 @@ class ApplicationMailer < ActionMailer::Base
 
   def roadie_options
     # This lets us specify assets using relative paths in email templates
-    super.merge(url_options: { host: URI(main_app.root_url).host })
+    url = URI(main_app.root_url)
+    super.merge(url_options: { host: url.host, port: url.port })
   end
 end

app/views/layouts/mailer.html.haml

@@ -16,7 +16,7 @@
               %table{:bgcolor => "#f2f2f2"}
                 %tr
                   %td
-                    %img{src: ContentConfig.url_for(:footer_logo), width: "144", height: "50"}/
+                    = image_tag ContentConfig.url_for(:footer_logo), width: "144", height: "50"
                   %td{:align => "right"}
                     %h6.collapse
                       = Spree::Config[:site_name]

config/environments/development.rb

@@ -69,8 +69,7 @@ Openfoodnetwork::Application.configure do
 
   # Show emails using Letter Opener
   config.action_mailer.delivery_method = :letter_opener
-  config.action_mailer.default_url_options = { host: "0.0.0.0:3000" }
-  config.action_mailer.asset_host = "http://localhost:3000"
+  config.action_mailer.default_url_options = { host: "localhost:3000" }
 
   config.log_level = ENV.fetch("DEV_LOG_LEVEL", :debug)
 end