[DONE] Fanfounet / webtv speaker app

[fanfounet]

Before sending your pull request, make sure the following are done:

• [x] You have read our contribution guidelines.
• [x] Your PR targets the develop branch.
• [x] The title of your PR starts with [WIP] or [DONE].

ToDo

• [x] Create Speaker app
• [x] Speaker model
• [x] Admin app
• [x] USE_SPEAKER setting
• [x] add view in completion
• [x] add edit speakers
• [x] show speakers in video
• [x] test
• [x] langs

[AymericJak]

Et pourquoi avoir mis USE_SPEAKER à False ? Plutôt qu'à true ?

[fanfounet]

Merci Aymeric pour ces premières remarques. j'ai corrigé les petits trucs ! Puis j'ai fait comme toi pour le quiz dans la partie des jobs. C'est drôle j'ai bien le dossier migrations avec le fichier init mais ça ne doit pas le pusher étrange. Pour ta première question, oui un speaker est bien un contributeur (contributeur est plus large car il y a les auteurs, les réalisateurs etc), mais le but est de laisser l'app contributeur qui gère différemment l'association d'un contributeur à une vidéo, donc pour ne rien casser l'idée est de laisser contributeur et ajouter speaker qui à mon sens est plus utile dans un contexte webtv (voir l'issue). Du fait je pense qu'il faut mettre USE_SPEAKER à false car par défaut on laisse contributeur et on ajouter speaker si besoin de cet usage

[AymericJak]

Pour ce qui est du Speaker OK. Ça évitera de tout casser. Puis éventuellement proposer une fusion / struture plus optimisée dans le futur.

Pour ce qui est du __init__.py, c'est normal que ça ne s'ajoute pas en faisant un git add. Il fait faire : git add --force lien/du/fichier

[ptitloup]

Hello, on sépare les contributeurs des intervenants dans Pod. Les contributeurs sont ceux qui ont participé à la conception tans que les intervenants sont ceux filmés ! :) De plus, d'un point de vue technique, on a choisit de séparer les 2 pour faciliter le dev et la maintenance.

[AymericJak]

Autant pour moi, il y a le speaker manager !!!

[AymericJak]

Liste de petits changements vu en visio :

• title sur les boutons dans la page Speaker manager
• 

Modifier le title pour mettre le nom prénom et job.

• Rendre possible la recherche avec le firstname
• Speaker manager : Modifier le bouton éditer et supprimer (title)
• Speaker manager remplacer les icone dans le <th> par 'Actions'
• Modifier style de la modale 'Add speaker' <div class="d-flex align-items-center">
• Ajouter dans le SpeakerForm

  def __init__(self, *args, **kwargs):
      """Init method."""
      super(SpeakerForm, self).__init__(*args, **kwargs)
      self.fields = add_placeholder_and_asterisk(self.fields)

[fanfounet]

J'ai passé la PR en DONE après avoir corrigé les remarques d'Aymeric. Question css vou préférez quoi ?

[Badatos]

J'ai passé la PR en DONE après avoir corrigé les remarques d'Aymeric. Question css vou préférez quoi ?

Faudrait demander à Apolline ^^

[AymericJak]

J'ai passé la PR en DONE après avoir corrigé les remarques d'Aymeric. Question css vou préférez quoi ?

L'option du haut pour moi.

[gitguardian[bot]]

️✅ There are no secrets present in this pull request anymore.

If these secrets were true positive and are still valid, we highly recommend you to revoke them. Once a secret has been leaked into a git repository, you should consider it compromised, even if it was deleted immediately. Find here more information about risks.

---

^{_{🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.}}