Merge pull request #2463 from betagouv/improve-2364

Improve #2364

app/controllers/new_user/feedbacks_controller.rb

@@ -1,6 +1,6 @@
 class NewUser::FeedbacksController < ApplicationController
   def create
     current_user.feedbacks.create!(mark: params[:mark])
-    flash.notice = "Merci de votre retour"
+    flash.notice = "Merci de votre retour, si vous souhaitez nous en dire plus, n'hésitez pas à <a href='mailto:#{CONTACT_EMAIL}' target='_blank'>nous contacter par email</a>."
   end
 end

app/views/new_user/dossiers/index.html.haml

@@ -54,9 +54,9 @@
                   = dossier.updated_at.localtime.strftime("%d/%m/%Y")
     = paginate(@dossiers)
 
-    - if current_user.feedbacks.empty?
+    - if current_user.feedbacks.empty? || current_user.feedbacks.last.created_at < 1.month.ago
       #user-satisfaction
-        %h3 Que pensez-vous de ce service ?
+        %h3 Que pensez-vous de la facilité d'utilisation de ce service ?
         .icons
           = link_to feedback_path(mark: 0), data: { remote: true, method: :post } do
             %span.icon.frown

app/views/new_user/feedbacks/create.js.erb

@@ -1,2 +1,3 @@
+window.scroll({ top: 0, left: 0, behavior: 'smooth' });
 <%= remove_element('#user-satisfaction') %>
 <%= render_flash %>

spec/views/new_user/dossiers/index.html.haml_spec.rb

@@ -74,7 +74,7 @@ describe 'new_user/dossiers/index.html.haml', type: :view do
 
   context "quand le user n'a aucun feedback" do
     it "affiche le formulaire de satisfaction" do
-      expect(rendered).to have_selector('#user-satisfaction', text: 'Que pensez-vous de ce service ?')
+      expect(rendered).to have_selector('#user-satisfaction', text: 'Que pensez-vous de la facilité d\'utilisation de ce service ?')
     end
   end