Missing inheritFilters field for Collection for InitialData

[rsat]

Describe the bug

When importing data or populating there is no option to set inheritFilters to false. Probably this was kept from before the inheritFilters was introduced

vendure/packages/core/src/data-import/types.ts

Lines 23 to 33 in 218561f

	export interface CollectionDefinition {
	name: string;
	description?: string;
	slug?: string;
	private?: boolean;
	filters?: CollectionFilterDefinition[];
	parentName?: string;
	assetPaths?: string[];
	}

vendure/packages/core/src/data-import/providers/populator/populator.ts

Lines 137 to 152 in 218561f

	const collection = await this.collectionService.create(ctx, {
	translations: [
	{
	languageCode: ctx.languageCode,
	name: collectionDef.name,
	description: collectionDef.description || '',
	slug: collectionDef.slug ?? collectionDef.name,
	},
	],
	isPrivate: collectionDef.private || false,
	parentId,
	assetIds: assets.map(a => a.id.toString()),
	featuredAssetId: assets.length ? assets[0].id.toString() : undefined,
	filters,
	});
	collectionMap.set(collectionDef.name, collection);

To Reproduce
Steps to reproduce the behavior:

1. Import data with collections
2. All collections are set to InheritFilters to true by default when importing data.

Expected behavior

Imported Collections should keep setting inheritFilters to true by default but should be able to set as false if desired.

Environment (please complete the following information):

• @vendure/core version: 2.1.1
• Nodejs version 16.20
• Database (mysql/postgres etc): postgres

Additional context
Could create the pull request with the fix for approval

[michaelbromley]

Hi Ricardo!

You're right - the format has not been updated when we added the inheritFilters toggle in v2. If you are willing to make a PR then that would be very welcome!

[bhardwajakshit]

@michaelbromley, Raised a PR for this: #2534

[michaelbromley]

@bhardwajakshit thanks for the fix!