Skip to content

Commit 8bac99a

Browse files
committed
Make manifest link optional out the gate as well
And point to having to turn that on in the routes.rb file.
1 parent 63be97b commit 8bac99a

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

railties/lib/rails/generators/rails/app/templates/app/views/layouts/application.html.erb.tt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99

1010
<%%= yield :head %>
1111

12-
<link rel="manifest" href="/manifest.json">
12+
<!-- link rel="manifest" href="/manifest.json" -->
1313
<link rel="icon" href="/icon.png" type="image/png">
1414
<link rel="icon" href="/icon.svg" type="image/svg+xml">
1515
<link rel="apple-touch-icon" href="/icon.png">

railties/lib/rails/generators/rails/app/templates/config/routes.rb.tt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ Rails.application.routes.draw do
66
get "up" => "rails/health#show", as: :rails_health_check
77

88
<%- unless options.api? -%>
9-
# Render dynamic PWA files from app/views/pwa/*
9+
# Render dynamic PWA files from app/views/pwa/* (remember to link manifest in application.html.erb)
1010
# get "service-worker" => "rails/pwa#service_worker", as: :pwa_service_worker
1111
# get "manifest" => "rails/pwa#manifest", as: :pwa_manifest
1212

0 commit comments

Comments
 (0)