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

refactor(Recherche): basculer la recherche sémantique dans le moteur de recherche existant #1115

Merged
merged 3 commits into from
Mar 29, 2024

Conversation

raphodn
Copy link
Contributor

@raphodn raphodn commented Mar 4, 2024

Quoi ?

On a déjà un SiaeFilterForm qui sert à beaucoup d'endroit dans notre application (recherche, téléchargement...).

Dans la PR #1073 qui introduit la nouvelle recherche "sémantique" par mot clé, on avait créé un formulaire séparé (SiaeSemanticForm). Les soucis que cela posait :

  • pas de bouton "partager la liste"
  • pas de changement dans l'url
  • pas de stats avec le nombre de résultats
  • du code dupliqué (template par exemple)

Ce refactoring permet de faire cohabiter tout le monde :)

Comment ?

  • supprimé la vue SiaeSemanticSearchResultsView
  • supprimé SiaeSemanticForm et basculé tout dans SiaeFilterForm
  • supprimé le template dédié semantic_search_results.html et basculé tout dans search_results.html
  • enlevé l'ordering si c'est une recherche sémantique
  • réparé les tests

Todo

  • éviter la pagination ?

@raphodn raphodn marked this pull request as draft March 5, 2024 08:14
@raphodn raphodn force-pushed the raphodn/semantic-search-refactoring branch from 47766f6 to 52c8db1 Compare March 27, 2024 09:25
@raphodn raphodn changed the title refactor(Recherche): basculer la recherche par mot clé dans le moteur de recherche existant refactor(Recherche): basculer la recherche sémantique dans le moteur de recherche existant Mar 27, 2024
@raphodn raphodn force-pushed the raphodn/semantic-search-refactoring branch from 52c8db1 to 25ae723 Compare March 27, 2024 09:36
@raphodn raphodn marked this pull request as ready for review March 27, 2024 09:41
Copy link
Contributor

@SebastienReuiller SebastienReuiller left a comment

Choose a reason for hiding this comment

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

LGTM ! Efficace, bravo !

@raphodn raphodn merged commit 7d78e1f into master Mar 29, 2024
5 checks passed
@raphodn raphodn deleted the raphodn/semantic-search-refactoring branch March 29, 2024 08:23
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.

2 participants