search

ToulouseDataViz / website

L'espace de dévelopment du site web toulouse-dataviz.fr
2 stars 2 forks source link

Delete unused hackaviz.js page #111

Closed emaulandi closed 2 years ago

emaulandi commented 2 years ago

closes #104

Il faut juste changer le contenu de home_content : https://github.com/ToulouseDataViz/website/blob/main/src/data/home_content.csv (fait dans le cadre de la mise à jour des soumissions 2021)

Les pages, ce n'est que de la présentation, il ne faut pas lui faire faire de la logique de redirection. Le fonctionnement actuel n'est pas bon !

Je vais faire un plugin qui va créer la page /hackaviz -> il n'y a pas besoin de page. Il ne faut pas avoir de page hackaviz.js. Le chemin pour accéder à une page existante depuis le menu ou les blocs de l'accueil est défini dans home_content, aucun besoin de faire une redirection au runtime du site.

vvivanloc commented 2 years ago

Il faut mettre à jour le footer pour qu'il aille aussi chercher l'url depuis le csv. 😕

emaulandi commented 2 years ago

Dans les différentes pages où il est utilisé, le composant prend en props l'année choisie, ou est-ce qu'il y a un problème ?

emaulandi commented 2 years ago

Ha ici https://github.com/ToulouseDataViz/website/blob/803a4db8bdd3e7a791780bcd5b93393bb7194a20/src/components/HackavizFooter.js#L16 ?

Après on peut aller aux éditions précédentes, donc on peut juste enlever ce button pour le moment.

vvivanloc commented 2 years ago

Ha ici

https://github.com/ToulouseDataViz/website/blob/803a4db8bdd3e7a791780bcd5b93393bb7194a20/src/components/HackavizFooter.js#L16 ?

Après on peut aller aux éditions précédentes, donc on peut juste enlever ce button pour le moment.

Bah vi 😀

vvivanloc commented 2 years ago

ok, j'ai confondu la page hackaviz.js avec le composant... Qu'ils aient le même nom ne m'aide pas :confounded: