Merge branch 'master' of github.com:noesya/osuny

app/controllers/server/universities_controller.rb

@@ -57,6 +57,6 @@ class Server::UniversitiesController < Server::ApplicationController
   end
 
   def university_params
-    params.require(:university).permit(:name, :address, :zipcode, :city, :country, :private, :identifier, :logo, :logo_delete)
+    params.require(:university).permit(:name, :address, :zipcode, :city, :country, :private, :identifier, :logo, :logo_delete, :sms_sender_name)
   end
 end

app/models/university.rb

@@ -11,14 +11,12 @@
 #  mail_from_name    :string
 #  name              :string
 #  private           :boolean
+#  sms_sender_name   :string
 #  zipcode           :string
 #  created_at        :datetime         not null
 #  updated_at        :datetime         not null
 #
 class University < ApplicationRecord
-  validates_presence_of :name
-  scope :ordered, -> { order(:name) }
-
   include WithCommunication
   include WithEducation
   include WithIdentifier
@@ -27,6 +25,11 @@ class University < ApplicationRecord
 
   has_one_attached_deletable :logo
 
+  validates_presence_of :name
+  validates :sms_sender_name, presence: true, length: { maximum: 11 }
+
+  scope :ordered, -> { order(:name) }
+
   def to_s
     "#{name}"
   end

app/services/sendinblue/sms_service.rb

@@ -11,7 +11,7 @@ module Sendinblue
     def self.send_message(user, message)
       api_instance = SibApiV3Sdk::TransactionalSMSApi.new
       send_transac_sms = SibApiV3Sdk::SendTransacSms.new(
-        sender: user.university.name,
+        sender: user.university.sms_sender_name,
         recipient: user.mobile_phone,
         content: message
       )

app/views/server/universities/_form.html.erb

@@ -4,6 +4,7 @@
       <%= f.input :name %>
       <%= f.input :identifier %>
       <%= f.input :private %>
+      <%= f.input :sms_sender_name %>
     </div>
     <div class="col-md-4">
       <%= f.input :address %>

config/locales/fr.yml

@@ -14,6 +14,7 @@ fr:
         private: Etablissement privé
         public: Public
         public_or_private: Public/privé
+        sms_sender_name: Nom de l'expéditeur SMS
         url: 'URL'
         zipcode: Code postal
       user:
@@ -97,6 +98,8 @@ fr:
     error_notification:
       default_message: "Les erreurs ci-dessous empêchent la validation :"
     hints:
+      university:
+        sms_sender_name: "11 caractères maximum."
       user:
         mobile_phone: "Format international (+XX). En renseignant ce champ, vous acceptez de recevoir vos codes de double authentification par SMS."
   simple_form_password_with_hints:

db/migrate/20211019083328_add_sms_sender_name_to_universities.rb

+class AddSmsSenderNameToUniversities < ActiveRecord::Migration[6.1]
+  def change
+    add_column :universities, :sms_sender_name, :string
+  end
+end

db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 2021_10_18_154621) do
+ActiveRecord::Schema.define(version: 2021_10_19_083328) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "pgcrypto"
@@ -282,6 +282,7 @@ ActiveRecord::Schema.define(version: 2021_10_18_154621) do
     t.datetime "updated_at", precision: 6, null: false
     t.string "mail_from_name"
     t.string "mail_from_address"
+    t.string "sms_sender_name"
   end
 
   create_table "users", id: :uuid, default: -> { "gen_random_uuid()" }, force: :cascade do |t|

lib/tasks/app.rake

@@ -9,11 +9,15 @@ namespace :app do
   desc 'Fix things'
   task fix: :environment do
     language = Language.first
-    User.find_each { |u|
-      u.confirm
-      u.role ||= :visitor
-      u.language ||= language
-      u.save
+    User.find_each { |user|
+      user.confirm
+      user.role ||= :visitor
+      user.language ||= language
+      user.save
+    }
+    University.find_each { |university|
+      university.sms_sender_name ||= 'Osuny'
+      university.save
     }
   end