Refactor AccessionOverviewSection
AccessionOverviewSection is doing some extra processing to generate overviewsTerms: Map. The original purpose was to include other and missing as additional TermItems.
Refactor PropertiesCard:
- Rename it to
OverviewDisplay - Add properties
missing?: numberandother?: number. These should be passed fromAccessionOverviewSectionas<OverviewDisplay ... other={ ... } missing={ ... } /> - Add
{ (other ?? 0) > 0 && <Property ...for other, and one for missing - These two should not use the
propertyItemProps
<Property should probably check for number to use localized formatting:
{ typeof value === 'number' ? <Number value={ value as number } /> : value }
Edited by Matija Obreza