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.

Sign up for GitHub

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

Jump to bottom

Supprime l'ancien module de recherche #6248

Merged
merged 1 commit into from
Feb 19, 2022
Merged

Supprime l'ancien module de recherche #6248

merged 1 commit into from
Feb 19, 2022

Conversation

Situphen
Copy link
Member

@Situphen Situphen commented Feb 19, 2022 •
edited
Loading

Supprime l'ancien module de recherche, enfin le peu qu'il en restait.

QA : Normalement Github Actions suffit, mais vous pouvez vérifier le bon fonctionnement de la recherche si vous le souhaitez.

@Situphen Situphen marked this pull request as ready for review February 19, 2022 19:22
@coveralls
Copy link

coveralls commented Feb 19, 2022 •
edited
Loading

Coverage Status

Coverage remained the same at 87.162% when pulling a8d80a6 on Situphen:nettoyage into 2b2c62d on zestedesavoir:dev.

@Situphen Situphen added the C-Search Concerne la recherche (et Typesense) label Feb 19, 2022
philippemilink
philippemilink approved these changes Feb 19, 2022
View reviewed changes
Copy link
Member

@philippemilink philippemilink left a comment

Choose a reason for hiding this comment

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

OK ✔️

@philippemilink philippemilink merged commit 576ac13 into zestedesavoir:dev Feb 19, 2022
@Situphen Situphen deleted the nettoyage branch February 19, 2022 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@philippemilink philippemilink philippemilink approved these changes

Assignees
No one assigned
Labels
C-Search Concerne la recherche (et Typesense)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Situphen @coveralls @philippemilink