Add required fields to CrudActions type (#33809)

packages/js/data/src/crud/types.ts

@@ -23,24 +23,34 @@ export type ItemQuery = BaseQueryParams & {
 	[ key: string ]: unknown;
 };
 
-export type CrudActions< ResourceName, ItemType, MutableProperties > =
+type WithRequiredProperty< Type, Key extends keyof Type > = Type & {
+	[ Property in Key ]-?: Type[ Property ];
+};
+
+export type CrudActions<
+	ResourceName,
+	ItemType,
+	MutableProperties,
+	RequiredFields extends keyof MutableProperties | undefined = undefined
+> = MapActions<
+	{
+		create: ( query: Partial< ItemType > ) => Item;
+		update: ( query: Partial< ItemType > ) => Item;
+	},
+	ResourceName,
+	RequiredFields extends keyof MutableProperties
+		? WithRequiredProperty< Partial< MutableProperties >, RequiredFields >
+		: Partial< MutableProperties >,
+	Generator< unknown, ItemType >
+> &
 	MapActions<
 		{
-			create: ( query: ItemQuery ) => Item;
+			delete: ( id: IdType ) => Item;
-			update: ( query: ItemQuery ) => Item;
 		},
 		ResourceName,
-		MutableProperties,
+		IdType,
 		Generator< unknown, ItemType >
-	> &
+	>;
-		MapActions<
-			{
-				delete: ( id: IdType ) => Item;
-			},
-			ResourceName,
-			IdType,
-			Generator< unknown, ItemType >
-		>;
 
 export type CrudSelectors<
 	ResourceName,

packages/js/data/src/product-shipping-classes/types.ts

@@ -26,14 +26,13 @@ type Query = BaseQueryParams< keyof ProductShippingClass > & {
 
 type ReadOnlyProperties = 'id';
 
-type MutableProperties = Partial<
+type MutableProperties = Omit< ProductShippingClass, ReadOnlyProperties >;
-	Omit< ProductShippingClass, ReadOnlyProperties >
->;
 
 type ProductShippingClassActions = CrudActions<
 	'ProductShippingClass',
 	ProductShippingClass,
-	MutableProperties
+	MutableProperties,
+	'name'
 >;
 
 export type ProductShippingClassSelectors = CrudSelectors<