Merge pull request 'change onClick prop to optional in IContextMenuItem' (#17) from feature/fix-context-menu-item into develop

TimofeyBoyko · TimofeyBoyko · commit 500829bc43bf · 2025-09-24T14:33:50.000Z
Reviewed-on: https://git.onlyoffice.com/ONLYOFFICE/docspace-plugin-sdk/pulls/17 Reviewed-by: Timofey Boyko <timofey.boyko@onlyoffice.com>
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -5,6 +5,7 @@
 ## Changed
 
 - Separate security for file in another enum FilesSecurity
+- change onClick prop to optional in IContextMenuItem
 
 ## Added
 
diff --git a/src/interfaces/items/IContextMenuItem.ts b/src/interfaces/items/IContextMenuItem.ts
@@ -125,7 +125,7 @@ export interface IContextMenuItem {
    * A function that takes the file/folder/room id as an argument. This function can be asynchronous
    *
    */
-  onClick: (id: number) => Promise<IMessage> | IMessage | void;
+  onClick?: (id: number) => Promise<IMessage> | IMessage | void;
 
   /**
    * Specifies elements as submenus.

Original file line number	Diff line number	Diff line change
`@@ -125,7 +125,7 @@ export interface IContextMenuItem {`
`125`	`125`	`* A function that takes the file/folder/room id as an argument. This function can be asynchronous`
`126`	`126`	`*`
`127`	`127`	`*/`
`128`		`- onClick: (id: number) => Promise<IMessage> \| IMessage \| void;`
	`128`	`+ onClick?: (id: number) => Promise<IMessage> \| IMessage \| void;`
`129`	`129`
`130`	`130`	`/**`
`131`	`131`	`* Specifies elements as submenus.`