Skip to content

implement admin/agents#index search

Vincent Agnano requested to merge feature/admin-agents-search into master

Created by: adipasquale

https://trello.com/c/hoyaICEM/1051-agent-pouvoir-chercher-un-agent-dans-la-liste-vos-agents

Screenshots

stitched_2020_09_09-14_57_28 Screenshot_2020-09-09_at_15 02 00

Notes

  • Je me suis basé sur le modèle de la recherche admin/users#index . Mais j'ai décidé de le faire sans AJAX, je ne trouve pas que ca apporte grand chose et en plus c'est a moitié cassé (par ex l'url n'est pas mise a jour lorsqu'on fait une recherche sur la page admin/users#index).

  • je ne filtre que la liste des agents complets a gauche, pas la liste des agents invités - elle sera probablement plus courte

  • j'ai fait des petites retouches de layout :

    • les tableaux prennent toute la largeur des cards,
    • on utilise un vrai card-header a droite dans les invitations plutot qu'une row de table hacky et pas belle
    • le champ de recherche peut etre vidé et soumis pour reinitialiser
    • j'affiche un compteur d'agents s'il y a plus d'une page en dessous de la pagination
    • tableau plus responsive avec break-word-all sur l'email
  • J'ai ensuite harmonisé tout ça avec la recherche sur admin/users#index pour que ca rester cohérent : enlever l'AJAX + retravaillé le layout pour que le tableau prenne toute la largeur

autres

pour creer des seeds sur la review app :

30.times do 
  Agent.new(
    email: Faker::Internet.email,
    role: :user,
    first_name: Faker::Name.first_name,
    last_name: Faker::Name.last_name,
    password: "123456",
    service: Service.secretariat,
    organisations: [Organisation.find(1)],
    invitation_accepted_at: 10.days.ago
  ).tap { _1.skip_confirmation! }.save!
end

Merge request reports