feat(ui-drilldown): make Drilldown.Page handle optional Options

INSTUI-4688

Author: ToMESSKa

packages/ui-drilldown/src/Drilldown/DrilldownPage/props.ts

@@ -39,7 +39,7 @@ import DrilldownSeparator from '../DrilldownSeparator'
 import type { OptionChild, SeparatorChild, GroupChild } from '../props'
 import { Renderable } from '@instructure/shared-types'
 
-type PageChildren = GroupChild | OptionChild | SeparatorChild
+type PageChildren = GroupChild | OptionChild | SeparatorChild | null | false
 
 type DrilldownPageOwnProps = {
   id: string

packages/ui-drilldown/src/Drilldown/index.tsx

@@ -528,15 +528,17 @@ class Drilldown extends Component<DrilldownProps, DrilldownState> {
     const childMap = new Map<string, boolean>()
 
     for (const child of children) {
-      if (!childMap.has(child.props.id)) {
-        childMap.set(child.props.id, true)
-      } else {
-        warn(
-          false,
-          `Duplicate id: "${child.props.id}"! Make sure all options have unique ids, otherwise they won't be rendered.`
-        )
+      if (child && typeof child === 'object' && child.props?.id) {
+        if (!childMap.has(child.props.id)) {
+          childMap.set(child.props.id, true)
+        } else {
+          warn(
+            false,
+            `Duplicate id: "${child.props.id}"! Make sure all options have unique ids, otherwise they won't be rendered.`
+          )
 
-        return (containsDuplicate = true)
+          return (containsDuplicate = true)
+        }
       }
     }