Skip to content

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:

  1. Rename it to OverviewDisplay
  2. Add properties missing?: number and other?: number. These should be passed from AccessionOverviewSection as <OverviewDisplay ... other={ ... } missing={ ... } />
  3. Add { (other ?? 0) > 0 && <Property ... for other, and one for missing
  4. 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
To upload designs, you'll need to enable LFS and have an admin enable hashed storage. More information