Skip to content

Add APIMixin.endpoint_with_id#14

Merged
pederhan merged 1 commit intomainfrom
endpoint-overrides
Apr 20, 2026
Merged

Add APIMixin.endpoint_with_id#14
pederhan merged 1 commit intomainfrom
endpoint-overrides

Conversation

@pederhan
Copy link
Copy Markdown
Member

@pederhan pederhan commented Apr 20, 2026

Adds an interface for overriding the default endpoint resolution for modification/deletion of resources: APIMixin.endpoint_with_id.

This lets us remove custom patch/delete methods and instead use APIMixin.{patch,delete} for everything by simply overriding endpoint_with_id where it is required.

@pederhan
Copy link
Copy Markdown
Member Author

Tested with mreg-cli:

No differences found between testsuite-result.json and new_testsuite_log.json

@pederhan pederhan merged commit 1079a9f into main Apr 20, 2026
8 checks passed
@pederhan pederhan deleted the endpoint-overrides branch April 20, 2026 09:41
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.

1 participant