datamade / django-councilmatic

:heartpulse: Django app providing core functions for *.councilmatic.org
http://councilmatic.org
MIT License
26 stars 16 forks source link

Refactor search-order #210

Closed reginafcompton closed 5 years ago

reginafcompton commented 6 years ago

This PR addresses a few search-related code smells:

  1. it simplifies the template logic for showing sort and order options;
  2. it uses the conventions of urllib for url parsing and encoding;
  3. it removes the full_text from the the Solr index (which contains RTF) and instead uses only the ocr_full_text.

@hancush - requesting your review....but @evz would welcome your feedback, too.

Relates to https://github.com/datamade/la-metro-councilmatic/issues/309