Skip to content

Add Open Zaak edit link#366

Merged
svenvandescheur merged 9 commits intomainfrom
322-als-een-joeri-wil-ik-een-linkje-naar-edit-in-open-zaak
Nov 7, 2025
Merged

Add Open Zaak edit link#366
svenvandescheur merged 9 commits intomainfrom
322-als-een-joeri-wil-ik-een-linkje-naar-edit-in-open-zaak

Conversation

@CharString
Copy link
Contributor

No description provided.

@CharString CharString linked an issue Nov 7, 2025 that may be closed by this pull request
@codecov-commenter
Copy link

codecov-commenter commented Nov 7, 2025

Codecov Report

❌ Patch coverage is 72.72727% with 21 lines in your changes missing coverage. Please review.
✅ Project coverage is 70.38%. Comparing base (15faf3d) to head (f5c203b).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...tend/src/components/related/RelatedObjectBadge.tsx 0.00% 9 Missing ⚠️
.../zaaktype/components/zaaktypetabs/ZaaktypeTabs.tsx 0.00% 7 Missing ⚠️
backend/src/openbeheer/types/_open_beheer.py 91.30% 2 Missing and 2 partials ⚠️
frontend/src/pages/zaaktype/zaaktype.action.ts 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #366      +/-   ##
==========================================
+ Coverage   70.27%   70.38%   +0.11%     
==========================================
  Files         168      168              
  Lines        5356     5420      +64     
  Branches      569      577       +8     
==========================================
+ Hits         3764     3815      +51     
- Misses       1553     1564      +11     
- Partials       39       41       +2     
Flag Coverage Δ
backend 90.23% <93.33%> (+0.08%) ⬆️
jest 27.88% <0.00%> (-0.14%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 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.

@CharString
Copy link
Contributor Author

@svenvandescheur Kun jij dit aan jouw Zaaktype link in de bovenhoek breien? En de CSS van de linkjes op de Overzicht tab fatsoeneren?

Roltypen worden daar niet vermeld, maar die ontsluiten we volledig, dus ik denk dat dat niet erg is.


def __post_init__(self):
super().__post_init__()
self.admin_url = _admin_url("zaaktypeobjecttype", self.uuid, self.url)

Choose a reason for hiding this comment

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

Suggested change
self.admin_url = _admin_url("zaaktypeobjecttype", self.uuid, self.url)
self.admin_url = _admin_url("zaakobjecttype", self.uuid, self.url)


def __post_init__(self):
super().__post_init__()
self.admin_url = _admin_url("zaaktypeobjecttype", self.uuid, self.url)

Choose a reason for hiding this comment

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

Suggested change
self.admin_url = _admin_url("zaaktypeobjecttype", self.uuid, self.url)
self.admin_url = _admin_url("zaakobjecttype", self.uuid, self.url)

@CharString CharString force-pushed the 322-als-een-joeri-wil-ik-een-linkje-naar-edit-in-open-zaak branch from 4e9bdcb to ab7720e Compare November 7, 2025 10:51
@svenvandescheur svenvandescheur force-pushed the 322-als-een-joeri-wil-ik-een-linkje-naar-edit-in-open-zaak branch from 82a134e to f5c203b Compare November 7, 2025 12:26
@svenvandescheur svenvandescheur merged commit ec2e9a0 into main Nov 7, 2025
16 checks passed
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.

Als een Joeri wil ik een linkje naar edit in open zaak

4 participants