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

[zep-13] Les billets sont absents de la recherche #4133

Closed
firm1 opened this issue Jan 5, 2017 · 9 comments
Closed

[zep-13] Les billets sont absents de la recherche #4133

firm1 opened this issue Jan 5, 2017 · 9 comments
Labels
S-BUG Corrige un problème

Comments

@firm1
Copy link
Contributor

firm1 commented Jan 5, 2017

Il m'est impossible de retrouver un billet en passant par la recherche du site (cases cochées ou non)

@firm1 firm1 added the S-BUG Corrige un problème label Jan 5, 2017
@pierre-24
Copy link
Member

pierre-24 commented Jan 5, 2017

"normal".

Je suis pas sur, vu comment la recherche est codée (puisque ladite recherche fait la différence entre article et tuto), que le moindre billet sois indexé. Ceci étant dit, ma PR sur Elasticsearch est quasi prête, et mon but (en tout cas c'est comme ça que je l'imagine) est d'attendre la ZEP-13 pour l'achever (justement pour tenir compte de cette histoire de "validatin"). Du coup, est ce que ça vaut la peine de ce casser le c*** pour qu'au final ma PR refasse le job ?

@vhf ?

@firm1
Copy link
Contributor Author

firm1 commented Jan 5, 2017

Ce n'est pas dramatique si les tribunes sortent sans la recherche dessus je pense. ça peut arriver dans une autre itération. Mais je tenais quand même à noter le souci dans une issue, parce que sinon c'est oublié

@pierre-24
Copy link
Member

pierre-24 commented Jan 5, 2017

J'me le met dans la liste de tickets de ma PR pour le garder de vue. (bordel, moi et le français, pour le moment ... ça veut rien dire!)

Je le met dans la liste des issues résolues par ma PR afin d'y jeter un coup d'oeil de temps à autre.

@pierre-24
Copy link
Member

pierre-24 commented Feb 5, 2017

Bon, au final, elastic est arrivé avant la ZEP-13. La question reste : on indexe les billets ? Tous? On discriminé sur la mise en avant ou pas ?

@vhf
Copy link
Contributor

vhf commented Feb 10, 2017

@pierre-24 question pour @FanJiyong à mon avis. :)

@artragis
Copy link
Member

artragis commented Apr 1, 2017

Question toujours d'actualité.

@pierre-24
Copy link
Member

D'autant si on touche au bout !

@vhf
Copy link
Contributor

vhf commented Apr 2, 2017

Ok alors on indexe les billets, mais uniquement ceux publiés, et uniquement leurs versions actuellement en ligne. Partons sur un boost de 2 sur les billets en avant, et 1 sur les autres.

@artragis
Copy link
Member

artragis commented Apr 2, 2017

Done, thx @pierre-24

@artragis artragis closed this as completed Apr 2, 2017
artragis pushed a commit to artragis/zds-site that referenced this issue Apr 10, 2017
artragis pushed a commit that referenced this issue Apr 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-BUG Corrige un problème
Projects
None yet
Development

No branches or pull requests

4 participants