Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BO - Export] Effet de bord session sur les filtres #3321

Merged
merged 3 commits into from
Nov 26, 2024

Conversation

sfinx13
Copy link
Collaborator

@sfinx13 sfinx13 commented Nov 21, 2024

Ticket

#3314

Description

Un effet de bord que je n'arrive pas à expliquer, quand on passe sur la liste des étiquettes pour faire un export, on se retrouve avec la session précédente. (Faut vraiment rafraîchir plusieurs fois la liste pour pour avoir la session à jour)

Pour rappel, la session est mise à jour via la requête ajax

Changements apportés

  • Stocker la session des filtre coté front via un cookie
  • Vérifier lors de l'export le cookie dans un premier temps puis la session

Pré-requis

Tests

  • Faire une recherche
  • J'affiche la page Etiquettes
  • Je clique sur Voir sur la liste des signalements avec étiquettes
  • Je clique sur le bouton Exporter les résultats le nombre doit correspondre à la liste filtré

@sfinx13 sfinx13 force-pushed the bugfix/3314-issue-filters branch from bfe6764 to b5ae8a9 Compare November 21, 2024 09:20
sfinx13 added a commit that referenced this pull request Nov 21, 2024
@sfinx13 sfinx13 force-pushed the bugfix/3314-issue-filters branch from fbe81e9 to 321f0fa Compare November 21, 2024 09:30
@sfinx13 sfinx13 changed the title [BO - Export] Filtres non sauvegardés dans la page d'export lors d'un partage ou en partant de la page étiquette [BO - Export] Effet de bord session sur les filtres Nov 21, 2024
Copy link
Collaborator

@numew numew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lecture ok, mais j'ai pas réussi à reproduire le problème

@sfinx13
Copy link
Collaborator Author

sfinx13 commented Nov 21, 2024

Copy link
Collaborator

@hmeneuvrier hmeneuvrier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok pour moi lecture et tests, je fais le test sur develop pour voir

@sfinx13 sfinx13 marked this pull request as draft November 21, 2024 16:46
@sfinx13 sfinx13 marked this pull request as ready for review November 26, 2024 09:14
@sfinx13 sfinx13 merged commit 34d841e into develop Nov 26, 2024
3 checks passed
@sfinx13 sfinx13 deleted the bugfix/3314-issue-filters branch November 26, 2024 11:20
@sfinx13 sfinx13 mentioned this pull request Dec 4, 2024
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants