ae-utbm / sith

🌐 The website of the AE
https://ae-utbm.github.io/sith/
GNU General Public License v3.0
6 stars 4 forks source link

Complete webpack migration, introduction of tom select, better SAS moderation workflow, more ruff and bugfixes #898

Closed imperosol closed 3 days ago

imperosol commented 3 days ago

Webpack migration

Mission accomplie :tada: ChartJS et la Galaxy sont passées sous webpack. Toutes les deps JS sont maintenant gérées via webpack.

Egalement, un système d'enregistrement des composants par décorateur a été ajouté, pour faciliter et standardiser l'intégration de nos widgets JS dans nos templates Jinja (sur tous les navigateurs, y compris Safari de ses morts).

Mais c'est pas fini non plus. On va encore passer du temps à améliorer ça.

Tom Select

Select2 était un peu mieux que django-ajax-select, mais en vrai c'est toujours un peu chiant (notamment parce que ça nécessite encore de s'emmerder avec jquery). Maintenant que l'outillage JS du projet et un peu plus mature, on tente le passage à Tom-select, qui est pas très différent en terme d'UI et d'UX, mais qui est beaucoup plus léger et qui marche sans jquery.

Et au passage on règle un gros problème de XSS qu'on a remarqué dans notre wrapper autour de select2.

Modération du SAS

Maintenant, cliquer sur le bouton "demander le retrait" n'est plus suffisant pour demander la modération. Au lieu de directement effectuer la demande de retrait, on redirige vers un formulaire sur lequel l'utilisateur doit dire pourquoi il veut que la photo soit supprimée. Les admins SAS ont accès aux demandes de modération, avec leur date, leur auteur et la raison de la demande.

Comme ça, on évite les missclicks, et les admins peuvent faire la différence entre les demandes sérieuses et celles qui sont involontaires ou loufoques.

Outillage

Bugfixes

Autres

Optimisation de la page d'affichage des produits (résolution d'un N+1 queries, et réduction des espaces dans le template rendu).

imperosol commented 3 days ago

Alors. Le déploiement auto marche pas. Mais c'est en cours de réparation dans #893

J'ai déployé manuellement pour compenser. La version en taiste est à jour avec la branche.