Skip to content

Resolve "[Carto] Le même filtre s'affiche plusieurs fois - Regression"

Hugo NOUTS requested to merge 322-carto-le-meme-filtre-s-affiche-plusieurs-fois-regression into dev

What does this MR do and why?

Related to #322 (closed)

Screenshots or screen recordings

How to set up and validate locally (or on alpha)

Sur la page cartographie :

  • On peut mettre plusieurs paramètres identiques dans l'url, ça ne les dupplique plus dans les filtres
  • On peut ajouter / retirer / ajouter autant de fois qu'on veut un filtre, il n'apparait qu'une fois dans la liste après validation.
  • On peut bien naviguer sur les checkbox par tabulation et les actionner avec le bouton "espace".

MR acceptance checklist

To be completed by the chosen reviewer.

Quality Bugs - Code Smells

  • For the code that this change impacts, I believe that the automated tests validate functionality that is highly important to users. If the existing automated tests do not cover this functionality, I have added the necessary additional tests or I have added an issue to describe the automation testing gap and linked it to this MR.
  • I have made sure that the sonar quality coverage is up to standards.
  • I have considered the impact of this change on the front-end, back-end, and database portions of the system where appropriate and applied.
  • I have tested this MR in all supported browsers or determined that this testing is not needed.
  • I have confirmed that this change is backwards compatible across updates (migrate up needs a migrate down), or I have decided that this does not apply.

Performance, reliability and availability

  • I am confident that this MR does not harm performance, or I have asked a reviewer to help assess the performance impact.
  • I have considered the scalability risk based on future predicted growth.

Documentation

  • The MR is named after the desired squash commit to feed the changelog linked to the current milestone.
  • I have added/updated documentation (also updated if the changes feature a deprecation) or I have decided that documentation changes are not needed for this MR.

Security Security Rating

  • I have confirmed that if this MR does not contains any sensitive informations hidden in the changes.
Edited by Hugo NOUTS

Merge request reports