SciencesPo / isari

GNU Affero General Public License v3.0
1 stars 2 forks source link

Supprimer la query-string "organization" dans les liens "organisations" et "mon CV" #139

Open naholyr opened 7 years ago

naholyr commented 7 years ago

cf. #106

Après vérification le ?organization est assez compliqué à supprimer au final, trois solutions :

  • Le supprimer mais ça nécessite quelques heures et une remise à plat de pas mal de choses au niveau des permissions et de ce qu'il est nécessaire d'avoir comme infos pour les connaître
  • Lui affecter comme valeur une orga arbitraire parmi celles de l'utilisateur connecté
  • Lui affecter comme valeur l'ID d'une orga "factice" mais existant réellement dans la base (mais ça nécessitera sûrement de mettre des filtres par ailleurs dans les API, a priori seulement dans le listing des orgas)
  • Pour le moment j'implémente la 2e solution histoire d'avoir un "quick win", la solution simple fut d'ajouter ces liens avec ?organization=la-première-de-l'utilisateur ce qui n'est pas élégant.

La piste 1 est plus simple mais pollue les données.

paulgirard commented 7 years ago

La solution Quick win est ok mais pas tenable à termes. Le problème en terme d'usage est que la fiche people devrait s'afficher sans le tableau des effectifs. De la même manière que quand on fait new mais sans le bug #140

Si je comprends bien il y a besoin d'un refacto important notamment à cause des questions de droits/rôles.

Il faut qu'on discute de ce point avant de prendre une décision de laisser tel quel ou de dédier du temps à ce problème rapidement.

naholyr commented 7 years ago

La seule contrainte qu'on a niveau permissions/rôles c'est qu'une orga doit être fournie aux APIs REST si on n'est pas central. On peut maintenir cette contrainte tout en faisant disparaître l'orga de l'UI si on crée une nouvelle page :

On peut du coup faire un composant dédié qui gère ça, avec un template sans tableau ni viz, ça ne devrait pas être trop complexe.

naholyr commented 7 years ago

On a maintenant un /cv qui pointe vers le bon contenu. Les modifs finales sont assez légères, au passage on peut customiser le titre dans le header avec overrideOrganizationName, j'ai mis "Personal Space" en dur, on peut le vider ou autre…

Ready for review :)

naholyr commented 7 years ago

Pour le diff : https://github.com/SciencesPo/isari/compare/912210d...99581330

paulgirard commented 7 years ago

je la passe en pour le futur comme besoin de refacto à terme