DGNum/demarches-normaliennes
15
1
Fork 0
Code Issues 1 Pull requests Projects Releases Packages Wiki Activity

fix: few fixes after review

Browse source
This commit is contained in:
seb-by-ouidou 2023-12-21 17:34:31 +00:00 committed by seb-by-ouidou
parent c253e7dd65
commit e79de57fb5
6 changed files with 14 additions and 27 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

24
app/components/groupe_gestionnaire/groupe_gestionnaire_administrateurs/administrateur_component.rb
View file

@ -24,23 +24,19 @@ class GroupeGestionnaire::GroupeGestionnaireAdministrateurs::AdministrateurCompo
end
def remove_button
if is_there_at_least_another_active_admin? && @is_gestionnaire
button_to 'Retirer',
remove_gestionnaire_groupe_gestionnaire_administrateur_path(@groupe_gestionnaire, @administrateur),
method: :delete,
class: 'fr-btn fr-btn--sm fr-btn--tertiary',
form: { data: { turbo: true, turbo_confirm: "Retirer « #{@administrateur.email} » des administrateurs de « #{@groupe_gestionnaire.name} » ?" } }
end
button_to 'Retirer',
remove_gestionnaire_groupe_gestionnaire_administrateur_path(@groupe_gestionnaire, @administrateur),
method: :delete,
class: 'fr-btn fr-btn--sm fr-btn--tertiary',
form: { data: { turbo: true, turbo_confirm: "Retirer « #{@administrateur.email} » des administrateurs de « #{@groupe_gestionnaire.name} » ?" } }
end
def destroy_button
if is_there_at_least_another_active_admin?
button_to 'Supprimer',
gestionnaire_groupe_gestionnaire_administrateur_path(@groupe_gestionnaire, @administrateur),
method: :delete,
class: 'fr-btn fr-btn--sm fr-btn--tertiary',
form: { data: { turbo: true, turbo_confirm: "Supprimer « #{@administrateur.email} » en tant qu'administrateurs ?" } }
end
button_to 'Supprimer',
gestionnaire_groupe_gestionnaire_administrateur_path(@groupe_gestionnaire, @administrateur),
method: :delete,
class: 'fr-btn fr-btn--sm fr-btn--tertiary',
form: { data: { turbo: true, turbo_confirm: "Supprimer « #{@administrateur.email} » en tant qu'administrateurs ?" } }
end
def is_there_at_least_another_active_admin?

2
app/components/groupe_gestionnaire/groupe_gestionnaire_gestionnaires/gestionnaire_component.rb
View file

@ -24,7 +24,7 @@ class GroupeGestionnaire::GroupeGestionnaireGestionnaires::GestionnaireComponent
end
def remove_button
if is_there_at_least_another_active_admin? && @is_gestionnaire
if is_there_at_least_another_active_admin?
button_to 'Retirer',
gestionnaire_groupe_gestionnaire_gestionnaire_path(@groupe_gestionnaire, @gestionnaire),
method: :delete,

2
app/views/administrateurs/_main_navigation.html.haml
View file

@ -3,5 +3,7 @@
%li.fr-nav__item= link_to 'Mes démarches', admin_procedures_path, class:'fr-nav__link', 'aria-current': current_page?(controller: 'administrateurs/procedures', action: :index) ? 'page' : nil
- if Rails.application.config.ds_zonage_enabled
%li.fr-nav__item= link_to 'Toutes les démarches', all_admin_procedures_path(zone_ids: current_administrateur.zones), class:'fr-nav__link', 'aria-current': current_page?(all_admin_procedures_path) ? 'page' : nil
- if current_administrateur.groupe_gestionnaire_id
%li.fr-nav__item= link_to 'Mon groupe gestionnaire', admin_groupe_gestionnaire_path, class:'fr-nav__link', 'aria-current': current_page?(admin_groupe_gestionnaire_path) ? 'page' : nil
= render MainNavigation::AnnouncesLinkComponent.new

3
app/views/administrateurs/groupe_gestionnaire/gestionnaires.html.haml
View file

@ -13,6 +13,3 @@
%th= 'État'
%tbody#gestionnaires
= render(GroupeGestionnaire::GroupeGestionnaireGestionnaires::GestionnaireComponent.with_collection(@groupe_gestionnaire.gestionnaires.order('users.email'), groupe_gestionnaire: @groupe_gestionnaire, is_gestionnaire: false))
.fr-mt-4w
= #render 'add_admin_form', groupe_gestionnaire: @groupe_gestionnaire

8
app/views/administrateurs/procedures/_main_menu.html.haml
View file

@ -1,8 +0,0 @@
.fr-container
%nav#header-navigation.fr-nav{ role: 'navigation', 'aria-label': 'Menu principal administrateur' }
%ul.fr-nav__list
%li.fr-nav__item= link_to 'Mes démarches', admin_procedures_path, class:'fr-nav__link', 'aria-current': current_page?(controller: 'procedures', action: :index) ? 'true' : nil
- if Rails.application.config.ds_zonage_enabled
%li.fr-nav__item= link_to 'Toutes les démarches', all_admin_procedures_path(zone_ids: current_administrateur.zones), class:'fr-nav__link', 'aria-current': current_page?(all_admin_procedures_path) ? 'page' : nil
- if current_administrateur.groupe_gestionnaire_id
%li.fr-nav__item= link_to 'Mon groupe gestionnaire', admin_groupe_gestionnaire_path, class:'fr-nav__link', 'aria-current': current_page?(admin_groupe_gestionnaire_path) ? 'page' : nil

2
db/schema.rb
View file

@ -1194,4 +1194,4 @@ ActiveRecord::Schema[7.0].define(version: 2023_12_21_142727) do
add_foreign_key "users", "users", column: "requested_merge_into_id"
add_foreign_key "without_continuation_mails", "procedures"
add_foreign_key "zone_labels", "zones"
end
end