Merge branch 'production' into agd/dsfr-migration-invitation-form-and…

…-reset-password-agent

app/helpers/absences_helper.rb

@@ -1,5 +1,5 @@
 module AbsencesHelper
-  CALENDAR_BACKGROUND_COLOR = "rgba(127, 140, 141, 0.7)".freeze
+  CALENDAR_BACKGROUND_COLOR = "rgba(52, 57, 58, 0.7)".freeze
 
   def absence_tag(absence)
     if absence.expired?

app/services/concerns/users/creneaux_wizard_concern.rb

@@ -64,6 +64,10 @@ def services
     @services ||= matching_motifs.includes(:service).map(&:service).uniq.sort_by(&:name)
   end
 
+  def follow_up_motifs?
+    @follow_up_motifs ||= Motif.where(service: services).where.not(bookable_by: :agents).exists?(follow_up: true, deleted_at: nil)
+  end
+
   def next_availability_by_lieux
     return @next_availability_by_lieux if @next_availability_by_lieux
 

app/services/off_days.rb

@@ -65,6 +65,7 @@ def self.to_full_calendar_array
         start: jour_ferie.beginning_of_day.as_json,
         end: jour_ferie.end_of_day.as_json,
         backgroundColor: AbsencesHelper::CALENDAR_BACKGROUND_COLOR,
+        textColor: "white",
       }
     end
   end

app/views/search/_referent_booking_card.html.slim

@@ -10,8 +10,9 @@
             span>
             | vos rendez-vous
   - else
-    .card
-      .m-2
-        | Pour prendre un RDV de suivi avec un de vos agents référent,
-        span>
-        = link_to("connectez-vous", users_rdvs_path)
+    - if context.follow_up_motifs?
+      .card
+        .m-2
+          | Pour prendre un RDV de suivi avec un de vos agents référents,
+          span>
+          = link_to("connectez-vous", users_rdvs_path)

app/views/static_pages/contact.html.slim

@@ -22,13 +22,10 @@ span> Voir
     h3
       = mail_to current_domain.support_email,
         subject: "[Problème Usager]",
-        body: "Code postal: \n\rNom: \n\rProblème ou Question:", class: "btn btn-primary" do
-        span>
-          i.fa.fa-envelope
+        body: "Code postal: \n\rNom: \n\rProblème ou Question:", class: "fr-btn fr-btn--icon-left fr-icon-mail-fill" do
         | Contacter l'équipe technique
 
-    .alert.alert-warning.my-4.d-flex.align-items-center
-      i.fa.fa-exclamation-triangle>
+    .fr-alert.fr-alert--warning.fr-mb-3w
       .ml-2
         - if current_domain == Domain::RDV_MAIRIE
           ' Nous ne pouvons pas débloquer d'autres créneaux que ceux affichés,

app/views/users/users/_form.html.slim

@@ -16,15 +16,13 @@ ruby:
   - if rdv_wizard&.rdv&.requires_ants_predemande_number?
     = f.input :ants_pre_demande_number, required: true, hint: t("simple_form.hints.user.ants_pre_demande_number_html"), input_html: {style: "text-transform: uppercase;"}
   - if user.logged_once_with_franceconnect?
-    .alert.alert-info.d-flex.align-items-center
-      .mr-3
-        .fa.fa-info
-      div= I18n.t("users.franceconnect_frozen_fields")
+    .fr-alert.fr-alert--info.fr-mb-1w
+      = I18n.t("users.franceconnect_frozen_fields")
   - if current_user.signed_in_with_invitation_token?
     = f.input :email, disabled: user.email.present? && !user.email_changed?, required: true
   = f.input :phone_number, as: :tel, required: rdv_wizard&.motif&.phone?
   - if user.phone_number.present? && !user.phone_number_mobile?
-    .alert.alert-warning Vous ne recevrez pas de SMS avec ce numéro non-mobile
+    .fr-alert.fr-alert--warning.fr-mb-1w Vous ne recevrez pas de SMS avec ce numéro non-mobile
   div.mb-2 Préférences de notifications
   div= f.input :notify_by_email
   div= f.input :notify_by_sms

spec/controllers/admin/api/agenda/absences_controller_spec.rb

@@ -48,7 +48,7 @@
               "title" => absence.title,
               "start" => absence.starts_at.as_json,
               "end" => absence.ends_at.as_json,
-              "backgroundColor" => "rgba(127, 140, 141, 0.7)",
+              "backgroundColor" => "rgba(52, 57, 58, 0.7)",
               "url" => "/admin/organisations/#{organisation.id}/absences/#{absence.id}/edit",
             },
           ]

spec/features/agents/rdvs_collectifs/agent_can_create_rdv_collectif_spec.rb

@@ -18,6 +18,10 @@
     stub_netsize_ok
     travel_to(now)
     login_as(agent, scope: :agent)
+    # Depuis que les jours fériés sont affichés sur la journée complète dans le calendrier,
+    # cela peut nous empêcher de cliquer sur une plage horaire et générer une flaky.
+    # On les retire pour ce test
+    allow(OffDays).to receive(:to_full_calendar_array).and_return([])
     visit admin_organisation_agent_agenda_path(organisation, agent)
   end
 

spec/requests/prendre_rdv_spec.rb

@@ -45,9 +45,30 @@
         expect(response.body).to include("Sélectionnez le service avec qui vous voulez prendre un RDV")
       end
 
-      it "shows a hint to help find a rdv with a referent agent in case the user is looking for the service of a follow_up motifs" do
-        get root_path(departement: "75", city_code: "75056", latitude: "48.859", longitude: "2.347", address: "Paris 75001")
-        expect(response.body).to include("Pour prendre un RDV de suivi avec un de vos agents référent")
+      context "lorsqu’il n’y a pas de motif de suivi associé aux services" do
+        it "n’affiche pas l’invitation à se connecter pour prendre un RDV de suivi" do
+          get root_path(departement: "75", city_code: "75056", latitude: "48.859", longitude: "2.347", address: "Paris 75001")
+          expect(response.body).not_to include("Pour prendre un RDV de suivi avec un de vos agents référents")
+        end
+      end
+
+      context "lorsqu’il y a un motif de suivi associé aux services" do
+        let(:bookable_by) { :everyone }
+        let!(:follow_up_motif) { create(:motif, organisation: organisation, service: motif.service, follow_up: true, bookable_by:) }
+
+        it "affiche l’invitation à se connecter pour prendre un RDV de suivi" do
+          get root_path(departement: "75", city_code: "75056", latitude: "48.859", longitude: "2.347", address: "Paris 75001")
+          expect(response.body).to include("Pour prendre un RDV de suivi avec un de vos agents référents")
+        end
+
+        context "lorsque le motif est réservable que par un agent" do
+          let(:bookable_by) { :agents }
+
+          it "n’affiche pas l’invitation à se connecter" do
+            get root_path(departement: "75", city_code: "75056", latitude: "48.859", longitude: "2.347", address: "Paris 75001")
+            expect(response.body).not_to include("Pour prendre un RDV de suivi avec un de vos agents référents")
+          end
+        end
       end
     end
 
@@ -63,9 +84,30 @@
         expect(response.body).to include("Sélectionnez le motif de votre RDV")
       end
 
-      it "shows a hint to help find a rdv with a referent agent in case the user is looking for follow_up motifs" do
-        get root_path(departement: "75", city_code: "75056", latitude: "48.859", longitude: "2.347", address: "Paris 75001")
-        expect(response.body).to include("Pour prendre un RDV de suivi avec un de vos agents référent")
+      context "lorsqu’il n’y a pas de motif de suivi associé aux services" do
+        it "n’affiche pas l’invitation à se connecter" do
+          get root_path(departement: "75", city_code: "75056", latitude: "48.859", longitude: "2.347", address: "Paris 75001")
+          expect(response.body).not_to include("Pour prendre un RDV de suivi avec un de vos agents référents")
+        end
+      end
+
+      context "lorsqu’il y a un motif de suivi associé aux services" do
+        let(:bookable_by) { :everyone }
+        let!(:follow_up_motif) { create(:motif, organisation: organisation, service: motif.service, follow_up: true, bookable_by:) }
+
+        it "affiche l’invitation à se connecter pour prendre un RDV de suivi" do
+          get root_path(departement: "75", city_code: "75056", latitude: "48.859", longitude: "2.347", address: "Paris 75001")
+          expect(response.body).to include("Pour prendre un RDV de suivi avec un de vos agents référents")
+        end
+
+        context "lorsque le motif est réservable que par un agent" do
+          let(:bookable_by) { :agents }
+
+          it "n’affiche pas l’invitation à se connecter" do
+            get root_path(departement: "75", city_code: "75056", latitude: "48.859", longitude: "2.347", address: "Paris 75001")
+            expect(response.body).not_to include("Pour prendre un RDV de suivi avec un de vos agents référents")
+          end
+        end
       end
     end
   end

spec/services/off_days_spec.rb

@@ -41,8 +41,8 @@
   describe ".to_full_calendar_array" do
     it "returns the proper format for full calendar" do
       array = described_class.to_full_calendar_array
-      expect(array[0].keys).to match_array(%i[title start end backgroundColor])
-      expect(array[0][:backgroundColor]).to eq "rgba(127, 140, 141, 0.7)"
+      expect(array[0].keys).to match_array(%i[title start end backgroundColor textColor])
+      expect(array[0][:backgroundColor]).to eq "rgba(52, 57, 58, 0.7)"
     end
   end
 end