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

Recherche sur les articles (rebase de #1873) #2645

Merged
merged 4 commits into from
May 14, 2015

Conversation

pierre-24
Copy link
Member

Q R
Correction de bugs ? oui
Nouvelle Fonctionnalité ? non
Tickets (issues) concernés #1511

Rebase de la PR de poulp #1873

Note de QA

(bien que je l'aie déjà fait moi-même !!)

  • Créez un article contenant un terme que vous êtes sur de n'avoir employé nulle part, genre "baudruche", "rhinocéros" ou "australopithèque". Publiez le (l'indexation ne fonctionne que sur les articles publiés).
  • Installez solr comme la doc l'explique (c'est plus simple que ça n'y parait, et en fait très rapide, vérifiez juste bien d'avoir java avant)
  • Faites une recherche sur ce terme abscond et vérifiez que vous retombez bien sur l'article. Avec la bonne date de publication.

@pierre-24
Copy link
Member Author

D'où question pour @SpaceFox : techniquement, la v1.9 ne s'appellera pas comme ça. Du coup ... Comment j'y fait référence dans le update.md ?

@landscape-bot
Copy link

Code Health
Repository health increased by 0.00% when pulling 17a9170 on pierre-24:rebase_1873 into 4d29e17 on zestedesavoir:dev.

@pierre-24 pierre-24 added S-BUG Corrige un problème C-Back Concerne le back-end Django labels May 7, 2015
@Eskimon
Copy link
Contributor

Eskimon commented May 14, 2015

C'est un grand pas pour la QA : ICI TOUT EST BON !! Et je confirme, en fait solr c'est peanuts a installer si java est déjà présent et fonctionnel sur la machine !

Du coup rebase & bon pour merge !

@pierre-24
Copy link
Member Author

Rebased :)

@pierre-24
Copy link
Member Author

Par contre, je reste assez peu convaincu du fait que je fait la bonne chose dans le update.md. Cette nouvelle façon de numéroter a un énorme désavantage sur celle-là (ping @SpaceFox pour être sur)

@Eskimon
Copy link
Contributor

Eskimon commented May 14, 2015

Oui bonne remarque, je ne sais pas quoi te répondre... @SpaceFox je te laisse donc le merge en considérant ma QA valide :)

@landscape-bot
Copy link

Code Health
Repository health increased by 0.00% when pulling 421ccf3 on pierre-24:rebase_1873 into 7571e0d on zestedesavoir:dev.

1 similar comment
@landscape-bot
Copy link

Code Health
Repository health increased by 0.00% when pulling 421ccf3 on pierre-24:rebase_1873 into 7571e0d on zestedesavoir:dev.

@SpaceFox
Copy link
Contributor

Non, c'est bon.

SpaceFox added a commit that referenced this pull request May 14, 2015
Recherche sur les articles (rebase de #1873)
@SpaceFox SpaceFox merged commit c47d70e into zestedesavoir:dev May 14, 2015
@SpaceFox SpaceFox removed the S-BUG Corrige un problème label May 14, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Back Concerne le back-end Django
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants