Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed url fields urls. #281

Merged
merged 2 commits into from
Nov 29, 2024
Merged

Fixed url fields urls. #281

merged 2 commits into from
Nov 29, 2024

Conversation

bart-maykin
Copy link
Contributor

Fixes #254

Changes

Changed the view_name of the PersoonSerializer and BijlageSerializer to their own detail page

@bart-maykin bart-maykin requested a review from SonnyBA November 15, 2024 15:22
@bart-maykin bart-maykin force-pushed the feature/254-fix-api-url-fields branch from dbfcde0 to f75f41f Compare November 15, 2024 15:22
Copy link
Contributor

@SonnyBA SonnyBA left a comment

Choose a reason for hiding this comment

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

Could you create some tests that verify these changes are adding the intended behavior?

@bart-maykin bart-maykin requested a review from SonnyBA November 22, 2024 14:21
@bart-maykin bart-maykin merged commit 2d20f6b into master Nov 29, 2024
25 checks passed
@bart-maykin bart-maykin deleted the feature/254-fix-api-url-fields branch November 29, 2024 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants