Skip to content

feat: support for custom taxonomies for schema property to taxonomy - #1556

Open
michaelclaesson wants to merge 6 commits into
helsingborg-stad:mainfrom
michaelclaesson:feat/schema-taxonomy-filter
Open

feat: support for custom taxonomies for schema property to taxonomy #1556
michaelclaesson wants to merge 6 commits into
helsingborg-stad:mainfrom
michaelclaesson:feat/schema-taxonomy-filter

Conversation

@michaelclaesson

Copy link
Copy Markdown
Contributor

Add employment type to JobPosting taxonomies and add filter for injecting custom taxonomies from schema prop

@michaelclaesson
michaelclaesson force-pushed the feat/schema-taxonomy-filter branch from 1c59c84 to a5eaf38 Compare October 13, 2025 16:11
@michaelclaesson
michaelclaesson force-pushed the feat/schema-taxonomy-filter branch from a5eaf38 to 9352a39 Compare October 13, 2025 16:12
][$schemaType] ?? [];
];

$taxonomies = \apply_filters('Municipio/Schema/Taxonomy/' . $schemaType, $map[$schemaType] ?? [], $schemaType, $this->taxonomyFactory, $this->schemaToPostTypeResolver);

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use wpService.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will make tests pass.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suppose the mock test signature for the wp service has to be changed since we've added applyFilters. I'm not familiar with your test setup though so out on deep water.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

$this->wpService->applyFilters(

@sonarqubecloud

Copy link
Copy Markdown

@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants