Onglet satisfaction

[clairezed]

close #3451

Tests

• company_satisfaction.rb

  • scope shared
  • scope unshared
  • share
    • it doesnt share when no comment
    • it doesnt share when already shared
    • it shares only to done_users

• expert.rb

  • shared_satisfactions
  • shared_company_satisfactions
  • without_shared_satisfaction

• need.rb

  • shared_satisfactions

• shared_satisfaction.rb

  • scope seen
  • scope unseen
  • validates user_id, uniqueness: { scope: [:company_satisfaction_id, :expert_id] }

• user.rb

  • shared_satisfactions
  • shared_company_satisfactions

• shared_satisfactions_controller.rb

  • index
  • unseen
  • seen
  • mark_as_seen

Context :
- un besoin unseen user A
- 2 besoins seen user A
- 1 besoin unseen user B

• feature shared satisfaction
  • it displays comment
  • it displays mark_as_seen button
  • when button clicked, it changes counters

[clairezed]

@LucienMLD tu pourras faire ta revue avec le cache activé ? J'ai pas pensé à le faire de mon côté

[LucienMLD]

Il manque une traduction dans le message d'ActiveAdmin

Tous les utilisateurs vont se retrouver avec un onglet "Retours" et une bonne partie n'aura sans doute jamais rien dans cet onglet. Alors qu'il est mentionné dans le message que les entreprises peuvent donner leur avis avec quand le besoins est cloturé avec aide. Ca va peut-être paraitre bizarre pour certains qu'ils n'ont jamais rien alors qu'ils clôturent bien avec aide.

On sait pas trop à quoi correspond la date en haut à gauche des cartes, surtout s'il y a de l'écart avec le besoins. Peut être indiqué "Partagé le .." ?

[LucienMLD]

ça sert à quoi de mettre .size ?

[clairezed]

il me semble que c'est + rapide / déjà chargé en mémoire, un truc du genre. On avait commencé à le rajouter avec Julien, non ?

[LucienMLD]

je me suis trompé, c'etait pourquoi mettre to_a ?

[clairezed]

Tu veux revoir la revue de la revue, @LucienMLD ?

[LucienMLD]

Deuxième revu avec le cache activé ok
Il reste des trucs pas coché dans la Todo c'est normal ?

[clairezed]

Les cases non cochées : j'avais listé tous les tests possibles, mais me suis concentrée sur ceux qui, à mon sens, apportaient vraiment de la valeur