feat(logos): manage departments and organisations logos with scaleway #1845
Conversation
There was a problem hiding this comment.
Trop cool merci @qblanc 🙌 c'est beaucoup plus clair comme ça 💯 🔥 🚀 !!!
Je t'ai posé quelques questions.
Je suis pas sûr d'avoir compris cette partie de ta description:
J'ai donc du procéder à l'introduction d'un nouveau controller et de nouvelles routes, avec des customs policies pour réussir à résoudre tous ces problèmes tout en restant safe au niveau de la sécurité
J'ai changé l'implem aujourd'hui en la simplifiant, après discussion avec @Michaelvilleneuve (merci à lui pour les conseils 🙏). L'autre problème que j'avais résolu (pouvoir consulter les documents/logos en local) avait aussi été résolu dans sa PR sur l'envoi de liens d'exports, j'ai donc repris sa solution à la place. Je modifie la description de la PR pour plus de clarté |
There was a problem hiding this comment.
Merci pour les changements @qblanc et @Michaelvilleneuve pour la suggestion 🙌 !!
On pourrait rajouter des tests pour au moins un des deux models qui utilise ces validations customs
|
si tu n'as pas d'autres remarques @aminedhobb n'hésite pas à approve 🙏 |
closes #1712
closes #1788
closes #1808
Dans cette PR, je fais en sorte que les logos des départements et des organisations soient désormais servis pas scaleway plutôt que par la codebase de l'app. Pour rendre ça possible, je permet un upload et/ou une modification des logos de ces ressources depuis l'interface superadmin, et je retire le petit module de modification du nom du fichier du logo dans l'espace de config (non superadmin) d'une orga.
administrate-field-active_storagede continuer à utiliserrails_blob_path, malgré queactive_storage.draw_routesest set àfalse📋 To do après le déploiement :
logo_filenamede la tableorganisations