search

chairemobilite / evolution

Online survey platform for travel survey
MIT License
4 stars 9 forks source link

Fix demo_survey UI tests, type SectionConfig and remove hiddenInNav #789

Closed tahini closed 1 day ago

tahini commented 2 days ago

This PR verifies all sections completion and enablement in the SectionNav, thus fixing the UI tests failures since it was moved to typescript

It also completes the typing of the SectionConfig, adding the missing fields

The hiddenInNav property is removed, in favor of a parentSection's check, such that only sections without parents are in the navigation bar.

tahini commented 2 days ago

@samuel-duhaime ça affecte le générateur, j'ai testé avec longue distance et ça a l'air beau. Pourrais-tu voir si mettons avec dependance à l'auto si la présence du parentSection et le hiddenInNav vont vraiment de pair (hiddenInNav: true <=> parentSection is defined et hiddenInNav: false <=> parentSection undefined), ou si c'est possible/sensé d'avoir d'autres combinaisons?

samuel-duhaime commented 2 days ago

@samuel-duhaime ça affecte le générateur, j'ai testé avec longue distance et ça a l'air beau. Pourrais-tu voir si mettons avec dependance à l'auto si la présence du parentSection et le hiddenInNav vont vraiment de pair (hiddenInNav: true <=> parentSection is defined et hiddenInNav: false <=> parentSection undefined), ou si c'est possible/sensé d'avoir d'autres combinaisons?

@tahini Non, c'est beau avec ta nouvelle méthode. Je n'ai pas encore vu d'enquête qu'on a fait cela, c'est probablement un signe que ce n'est pas nécessaire.

samuel-duhaime commented 2 days ago

@tahini

Je crois qu'il y a un BREAKING CHANGE. Minimalement, les prochaines enquêtes mises à jour vont devoir faire 'yarn generateSurvey' car certaines importations ont changé.