fpdcc / document-search

Search and management interface for spatial documents at the Forest Preserves of Cook County.
2 stars 0 forks source link

Make sure to sort nulls last for all fields #44

Closed jeancochrane closed 4 years ago

jeancochrane commented 4 years ago

Overview

Currently, the behavior of the app is inconsistent in whether it displays null values first or last when sorting on specific fields. Update all search views to make sure that nulls are always sorted last.

Connects #37.

Testing Instructions

It's annoyingly rote, but the best way to test this PR is to visit the search view for each document type, test every sort option with the value ascending, and make sure for each option that the first value shown in the list isn't null. Note that there are two columns that will show nulls anyway: Survey:Number of sheets and Survey:Date, both of which have no non-null values in the current data.

jeancochrane commented 4 years ago

@derekeder Great catch, I'll quickly move description so that it's second on the list for Licenses.