Skip to content

Some component demos don't meet "strictNullChecks" requirements #3230

@seanforyou23

Description

@seanforyou23

I noticed that when a consuming typescript app uses the strictNullChecks compiler option, errors are thrown during build time. I'm seeing this issue in the following component demos in react-integration:

  • AccordionDemo/AccordionDemo.tsx
  • DonutUtilizationChartDemo/DonutUtilizationGreenStaticRightDemo.tsx
  • DonutUtilizationChartDemo/DonutUtilizationInvertedRightDemo.tsx
  • DonutUtilizationChartDemo/DonutUtilizationSimpleBottomDemo.tsx
  • DonutUtilizationChartDemo/DonutUtilizationSimpleDemo.tsx
  • DonutUtilizationChartDemo/DonutUtilizationSimpleRightDemo.tsx
  • DonutUtilizationChartDemo/DonutUtilizationSimpleRightGreenDemo.tsx
  • DonutUtilizationChartDemo/DonutUtilizationSmallDemo.tsx
  • DonutUtilizationChartDemo/DonutUtilizationSmallRightDemo.tsx
  • DonutUtilizationChartDemo/DonutUtilizationSmallStaticBottomDemo.tsx
  • DonutUtilizationChartDemo/DonutUtilizationSmallStaticDemo.tsx
  • DonutUtilizationChartDemo/DonutUtilizationSmallStaticRightDemo.tsx
  • DonutUtilizationChartDemo/DonutUtilizationStaticBottomDemo.tsx
  • DonutUtilizationChartDemo/DonutUtilizationStaticDemo.tsx
  • DonutUtilizationChartDemo/DonutUtilizationStaticInvertedRightDemo.tsx
  • DonutUtilizationChartDemo/DonutUtilizationStaticRightDemo.tsx
  • SelectDemo/SelectDemo.tsx
  • TableDemo/TableActionsDemo.tsx
  • TableDemo/TableSortableForCompoundExpandableDemo.tsx

It seems like a big list, but I think the problem for charts is the same one repeated in all of the chart demos. That said, I don't guess this will be a trivial fix.

Below is an example of how the error can manifest:

Screen Shot 2019-10-28 at 8 19 20 PM

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions