laws-africa / peachjam

Project Peach Jam
https://agp.africanlii.org
GNU General Public License v3.0
2 stars 0 forks source link

Document nature is now required #1991

Closed longhotsummer closed 1 week ago

longhotsummer commented 2 weeks ago

This ensures that all documents have a descriptive nature, even those without a subtype. For example, all Legislation objects now default to "Act". This means we can rely on the nature as a database-indexed description of the type of every document.

This makes it possible to facet on the nature. Before, we had to facet on a weird mixture of doc_type and nature.

In future, nature will be preferred over doc_type in search results. But we need to backfill the nature field in ES for that to work.

Other:

Example by-law listing:

image

Example general legislation listing:

image

Outstanding

longhotsummer commented 2 weeks ago

Closes https://github.com/laws-africa/peachjam/issues/815

github-actions[bot] commented 2 weeks ago

Test Results

48 tests   48 :white_check_mark:  11s :stopwatch: 13 suites   0 :zzz: 13 files     0 :x:

Results for commit ae59d108.

:recycle: This comment has been updated with latest results.

longhotsummer commented 1 week ago

@actlikewill this is ready for review

sentry-io[bot] commented 1 week ago

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

Did you find this useful? React with a 👍 or 👎