Skip to content

Feat/filter zaken on zaaktypen - #54

Open
mvdhoek1 wants to merge 1 commit into
mainfrom
feat/filter-zaken-on-zaaktypen
Open

Feat/filter zaken on zaaktypen#54
mvdhoek1 wants to merge 1 commit into
mainfrom
feat/filter-zaken-on-zaaktypen

Conversation

@mvdhoek1

@mvdhoek1 mvdhoek1 commented Aug 6, 2026

Copy link
Copy Markdown
Contributor
Screenshot 2026-08-06 at 08 57 22 Screenshot 2026-08-06 at 08 57 01

@mvdhoek1
mvdhoek1 marked this pull request as ready for review August 14, 2026 11:11
Comment thread src/WPCron/Events/PopulateZaaktypen.php Outdated
use function OWC\ZGW\apiClientManager;

/**
* Cron event that pre-fetches zaaktypen for all configured suppliers.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

dubbelop met de phpdoc hierboven?

use OWC\ZGW\Support\Collection;

/**
* Adapter for fetching zaaktypen to use as select options.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hier ook


private function get_zaaktype_timestamp( Zaaktype $zaaktype ): int
{
if ( ! $zaaktype->versiedatum instanceof DateTimeImmutable) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Jij weet beter wat die API teruggeeft denk ik, maar is een versienummer niet logischer dan een versiedatum?

@mvdhoek1
mvdhoek1 force-pushed the feat/filter-zaken-on-zaaktypen branch from bb1caa6 to 577f0b7 Compare September 1, 2026 06:32
@mvdhoek1
mvdhoek1 force-pushed the feat/filter-zaken-on-zaaktypen branch from 577f0b7 to 961c821 Compare September 1, 2026 08:15
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