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?: number
andother?: number
. These should be passed fromAccessionOverviewSection
as<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