Added better logging for our event system

src/api/useApiEventHandler.ts

@@ -1,5 +1,5 @@
 import { useEffect } from 'react'
-import { ApiRef } from './types/apiRef'
+import { ApiRef } from './types'
 import { useLogger } from '../logger'
 
 export function useApiEventHandler(
@@ -14,6 +14,9 @@ export function useApiEventHandler(
       return apiRef.current.subscribeEvent(eventName, handler)
     }
 
+    logger.warn(
+      `Event name or handler is missing therefore it isn't possible to subscribe. EventName: ${eventName}, Handler: ${handler}`,
+    )
     return undefined
   }, [apiRef, eventName, handler, logger])
 }

src/api/useApiFactory.ts

@@ -9,6 +9,8 @@ import { useLogger } from '../logger'
  * Initializes a new api instance
  * @param gridRootRef
  * @param apiRef
+ * @param theme
+ * @param selectionKey
  */
 export function useApiFactory(
   gridRootRef: React.RefObject<HTMLDivElement>,
@@ -21,18 +23,19 @@ export function useApiFactory(
 
   const publishEvent = useCallback(
     (name: string, ...args: any[]) => {
+      logger.debug(`Publishing event ${name}`)
       apiRef.current.emit(name, ...args)
     },
-    [apiRef],
+    [logger, apiRef],
   )
 
   const subscribeEvent = useCallback(
     (event: string, handler: (param: any) => void): (() => void) => {
-      logger.debug(`Binding ${event} event`)
+      logger.debug(`Subscribing to event: ${event}`)
       apiRef.current.on(event, handler)
       const api = apiRef.current
       return () => {
-        logger.debug(`Clearing ${event} event`)
+        logger.debug(`Removing event listener: ${event}`)
         api.removeListener(event, handler)
       }
     },
@@ -50,7 +53,7 @@ export function useApiFactory(
     const api = apiRef.current
 
     return () => {
-      logger.debug('Clearing all events listeners')
+      logger.debug('[Unmount] Clearing all events listeners')
       api.removeAllListeners()
     }
   }, [selectionKey, gridRootRef, apiRef, theme, logger])