Merge pull request #5932 from betagouv/sprockets-4

Mise à jour vers le processeur d'assets Sprockets v4 (#5932)

Gemfile

@@ -74,7 +74,6 @@ gem 'sib-api-v3-sdk'
 gem 'skylight'
 gem 'smart_listing'
 gem 'spreadsheet_architect'
-gem 'sprockets', '< 4'
 gem 'typhoeus'
 gem 'warden'
 gem 'webpacker'
@@ -112,7 +111,6 @@ group :development do
   gem 'rubocop-rspec-focused', require: false
   gem 'scss_lint', require: false
   gem 'web-console'
-  gem 'xray-rails'
 end
 
 group :development, :test do

Gemfile.lock

@@ -730,7 +730,7 @@ GEM
     spring (2.1.1)
     spring-commands-rspec (1.0.4)
       spring (>= 0.9.1)
-    sprockets (3.7.2)
+    sprockets (4.0.2)
       concurrent-ruby (~> 1.0)
       rack (> 1, < 3)
     sprockets-rails (3.2.2)
@@ -799,8 +799,6 @@ GEM
     websocket-extensions (0.1.5)
     xpath (3.2.0)
       nokogiri (~> 1.8)
-    xray-rails (0.3.2)
-      rails (>= 3.1.0)
     zeitwerk (2.4.2)
     zip_tricks (5.5.0)
     zipline (1.3.0)
@@ -918,7 +916,6 @@ DEPENDENCIES
   spreadsheet_architect
   spring
   spring-commands-rspec
-  sprockets (< 4)
   timecop
   typhoeus
   vcr
@@ -927,7 +924,6 @@ DEPENDENCIES
   webdrivers (~> 4.0)
   webmock
   webpacker
-  xray-rails
   zipline
   zxcvbn-ruby
 

app/assets/config/manifest.js

@@ -0,0 +1,5 @@
+//= link application.css
+//= link_tree ../images
+//= link_tree ../fonts
+//= link administrate/application.css
+//= link administrate/application.js

app/assets/stylesheets/application.scss

@@ -5,4 +5,3 @@
 // = require ./fonts
 // = require leaflet
 // = require_tree .
-// = stub ./print.scss

app/assets/stylesheets/print.scss

@@ -1,44 +1,45 @@
 @import "colors";
 @import "fonts";
 
-.new-header,
-.sub-header,
-footer {
+@media print {
+  .new-header,
+  .sub-header,
+  footer {
     display: none;
-}
+  }
 
-.print-header {
+  .print-header {
     display: block;
     margin-top: 30px;
     font-size: 30px;
     line-height: 50px;
     font-weight: bold;
-}
+  }
 
-body {
+  body {
     font-family: "Muli";
-}
+  }
 
-.subtitle {
+  .subtitle {
     margin-top: -20px;
     color: $dark-grey;
-}
+  }
 
-th,
-td {
+  th,
+  td {
     vertical-align: top;
-}
+  }
 
-th {
+  th {
     text-align: left;
 
     &.header-section {
       padding-top: 1.2em;
       font-size: 1.2em;
     }
-}
+  }
 
-.messagerie {
+  .messagerie {
     .messages-list {
       list-style: none;
       padding-left: 0;
@@ -52,12 +53,13 @@ th {
         display: none;
       }
     }
-}
+  }
 
-.message {
+  .message {
     margin-bottom: 40px;
-}
+  }
 
-.updated-at {
+  .updated-at {
     display: none;
+  }
 }

app/views/layouts/application.html.haml

@@ -20,7 +20,6 @@
     = preload_link_tag(asset_url("Muli-Bold.woff2"))
 
     = stylesheet_link_tag 'application', media: 'all'
-    = stylesheet_link_tag 'print', media: 'print'
 
     = Gon::Base.render_data(camel_case: true, init: true, nonce:  request.content_security_policy_nonce)
 

config/initializers/assets.rb

@@ -10,5 +10,6 @@ Rails.application.config.assets.paths << Rails.root.join('node_modules', '@reach
 Rails.application.config.assets.paths << Rails.root.join('node_modules', '@mapbox', 'mapbox-gl-draw', 'dist')
 
 # Precompile additional assets.
-# application.js, application.css, and all non-JS/CSS in app/assets folder are already added.
-Rails.application.config.assets.precompile += ['print.css']
+# application.js, application.css, and all non-JS/CSS in the app/assets
+# folder are already added.
+# Rails.application.config.assets.precompile += %w( admin.js admin.css )

config/initializers/sentry.rb

@@ -3,5 +3,5 @@ Sentry.init do |config|
   config.send_default_pii = false
   config.enabled_environments = ['production']
   config.breadcrumbs_logger = [:active_support_logger]
-  config.traces_sample_rate = 0.001
+  config.traces_sample_rate = ENV['SENTRY_ENABLED'] == 'enabled' ? 0.001 : nil
 end