Skip to content

chore(overige-objecten-api): migrate the overige-objecten-api to Strapi 5#1412

Open
AliKdhim87 wants to merge 3 commits intodevelopment-strapi-v5from
migrate-overige-objecten-api-strapi-v5
Open

chore(overige-objecten-api): migrate the overige-objecten-api to Strapi 5#1412
AliKdhim87 wants to merge 3 commits intodevelopment-strapi-v5from
migrate-overige-objecten-api-strapi-v5

Conversation

@AliKdhim87
Copy link
Copy Markdown
Collaborator

@AliKdhim87 AliKdhim87 commented Apr 6, 2026

Beschrijving

Deze pull request bevat de migratie van de overige-objecten-api naar Strapi v5 binnen het PDC-dashboard.

Daarnaast zijn er een aantal verbeteringen doorgevoerd:

  • Implementatie van code generation (codegen) voor betere type safety en één-op-één gegenereerde types
  • Verbeteringen in TypeScript typing voor robuustere en beter onderhoudbare code
  • Oplossing voor een ESModule issue binnen het ui package, zodat deze correct gebruikt kan worden door de overige-objecten-api
  • Integratie van de KISS preview met het PDC-dashboard

Hoe kun je dit testen?

Zonder Docker

  1. Environment variabelen genereren

    • Vanuit de root:
      pnpm run env
    • Dit genereert alleen .env bestanden voor:
      • pdc-dashboard
      • pdc-frontend
  2. Handmatig .env toevoegen voor overige-objecten-api

    • Maak een .env bestand aan in:

      apps/overige-objecten-api
      
    • Voeg de volgende variabelen toe:

      OVERIGE_OBJECTEN_API_PORT=4001
      STRAPI_PRIVATE_URL=http://localhost:1337
      OVERIGE_OBJECTEN_API_CORS=http://localhost:4001,http://localhost:4000,http://localhost:1337
      KISS_PREVIEW_TOKEN=<base64 waarde, moet overeenkomen met PDC-dashboard>
      STRAPI_API_TOKEN=<aanmaken via Strapi dashboard met read & write rechten>
      NODE_ENV=development
  3. Start de applicaties

    • Start:
      • pdc-dashboard
      • overige-objecten-api

Wat moet je testen?

1. KISS Preview functionaliteit

  • Ga naar producten en VACs in het PDC-dashboard
  • Controleer of de "KISS preview" knop zichtbaar is
  • Klik op de knop:
    • Je zou doorgestuurd moeten worden naar de overige-objecten-api
    • Preview van kennisartikelen en VACs moet correct werken

2. Create & Update functionaliteit

  • Test het aanmaken en updaten van:

    • VACs
    • Kennisartikelen
  • Gebruik hiervoor Swagger: http://localhost:4001/api/v2/api-docs/

  • Voorbeeldresponses zijn te vinden in:
    apps/overige-objecten-api/src/mocks

Met Docker

Je kunt de instructies volgen via de volgende link: https://github.com/frameless/strapi/tree/development-strapi-v5?tab=readme-ov-file#start-the-server-using-docker

#1376

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 6, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
cms-frameless-io Error Error Apr 16, 2026 9:41am
strapi-overige-objecten-api Ready Ready Preview, Comment Apr 16, 2026 9:41am
strapi-pdc-frontend Ready Ready Preview, Comment Apr 16, 2026 9:41am
tiptap Ready Ready Preview, Comment Apr 16, 2026 9:41am

Request Review

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 6, 2026

Codecov Report

❌ Patch coverage is 75.58248% with 262 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...ige-objecten-api/src/controllers/objects/update.ts 0.00% 103 Missing ⚠️
...ige-objecten-api/src/controllers/objects/create.ts 0.00% 76 Missing ⚠️
apps/overige-objecten-api/src/codegen.ts 0.00% 26 Missing ⚠️
apps/overige-objecten-api/gql/gql.ts 0.00% 18 Missing ⚠️
...rige-objecten-api/src/controllers/objects/index.ts 84.90% 8 Missing ⚠️
...pps/overige-objecten-api/src/utils/processData.tsx 91.11% 8 Missing ⚠️
...ige-objecten-api/src/utils/getPaginatedResponse.ts 0.00% 7 Missing ⚠️
apps/overige-objecten-api/src/service/object.ts 94.18% 5 Missing ⚠️
apps/overige-objecten-api/src/utils/getVacData.ts 87.50% 4 Missing ⚠️
apps/overige-objecten-api/src/utils/resolveDoc.ts 86.66% 2 Missing ⚠️
... and 5 more
Files with missing lines Coverage Δ
apps/overige-objecten-api/gql/graphql.ts 0.40% <ø> (ø)
...en-api/src/__mocks__/getStrapiKennisartikelData.ts 100.00% <100.00%> (ø)
.../src/__mocks__/getStrapiKennisartikelDataByUUID.ts 100.00% <100.00%> (ø)
...ige-objecten-api/src/__mocks__/getStrapiVacData.ts 100.00% <100.00%> (ø)
...jecten-api/src/__mocks__/getStrapiVacDataByUUID.ts 100.00% <100.00%> (ø)
...-objecten-api/src/__mocks__/kennisartikelObject.ts 100.00% <100.00%> (ø)
...-objecten-api/src/__mocks__/objectsResponseData.ts 100.00% <100.00%> (ø)
...ps/overige-objecten-api/src/__mocks__/vacObject.ts 100.00% <100.00%> (ø)
...-objecten-api/src/components/KennisartikelPage.tsx 79.22% <100.00%> (ø)
...s/overige-objecten-api/src/components/Markdown.tsx 96.82% <100.00%> (ø)
... and 48 more
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@AliKdhim87 AliKdhim87 force-pushed the migrate-overige-objecten-api-strapi-v5 branch from dd69ae4 to 4edbaa9 Compare April 13, 2026 14:41
@AliKdhim87 AliKdhim87 force-pushed the migrate-overige-objecten-api-strapi-v5 branch from 4edbaa9 to bb3cce1 Compare April 14, 2026 16:15
@AliKdhim87 AliKdhim87 force-pushed the migrate-overige-objecten-api-strapi-v5 branch from bb3cce1 to d5555dd Compare April 15, 2026 08:45
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.

Migreer Microservice overige-objecten-api naar Strapi v5 GraphQL

1 participant