Skip to content
This repository has been archived by the owner on Feb 22, 2024. It is now read-only.

A fairly bug-free version of the search #113

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

hc5
Copy link
Contributor

@hc5 hc5 commented Jan 24, 2012

Really really really sorry about the big commit but I couldn't think of a way to split it up logically, the good thing is that it doesn't modify any old behavior and barely changes existing code so it's quite easy to merge in.
Can't really find any bugs right now, but things TODO:

  • pagination of the results, currently only returns the first 10, but the offset parameter is coded in already, so it's easy to add.
  • Nicer preview text
  • Nicer search page in general
  • Better way of displaying the commits. They are already sorted by relevance, so it's just a GUI problem
  • Speed. I'm satisfactory with the search speed for now, but the indexing speed is still sort of slow

To "install" you have to syncdb then ./manage.py buildindex. This indexes all the existing documents. Subsequent document editions/creations will be indexed automatically

hc5 added 2 commits January 24, 2012 01:37
…ion and the search page is not very pretty at all. Speed is also something to be desired
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant