Sorting blanks depends on the column type

SEED-platform / seed

Standard Energy Efficiency Data (SEED) Platform™ is a web-based application that helps organizations easily manage data on the energy performance of large groups of buildings.

Other

106 stars 55 forks source link

Sorting blanks depends on the column type #4627

Open axelstudios opened 2 months ago

axelstudios commented 2 months ago

Describe the bug

Sorting canonical columns alphabetically puts empty values at the end
Sorting _extradata columns alphabetically puts empty values at the beginning

Sorting

Expected Behavior It should be consistent, not sure which we'd prefer

Actual Behavior It sorts inconsistently

Steps to Reproduce

Import a csv like:

PM Property ID,City,Extra
1,A,A
2,B,B
3,,

Sort the City and Extra columns to observe the issue

kflemin commented 2 months ago

go with the canonical field sort since it's already that way?