Skip to content

Refactor the search form and view

Erwan Rouchet requested to merge refactor-search into master

Closes #981 (closed), #1080 (closed)

  • Moved the search execution to the search view instead of the form
  • Promoted the corpus ID to a path parameter (/search/<id>/) and moved its management to the search view, greatly simplifying the search form
  • Removed the method prop on the search form, an ancient artifact dating from the ElasticSearch-based search
  • Decoupled the search form from the route similarly to !1357 (merged), fixing #981 (closed)
  • Fingers-crossed-driven development™: maybe by just rewriting everything I somehow fixed #1080 (closed). @mlbonhomme will have to test on a Mac!

Merge request reports