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

Deprecate the usage of all NestedSerializer classes #17143

Closed
jeremystretch opened this issue Aug 13, 2024 · 0 comments
Closed

Deprecate the usage of all NestedSerializer classes #17143

jeremystretch opened this issue Aug 13, 2024 · 0 comments
Assignees
Labels
status: accepted This issue has been accepted for implementation type: deprecation Removal of existing functionality or behavior
Milestone

Comments

@jeremystretch
Copy link
Member

Proposed Changes

Raise a deprecation warning whenever an obsolete nested serializer class (i.e. those defined in the nested_serializers module within each app) is imported.

Justification

As of NetBox v4.0 these are generally no longer used, and will be removed in a future release. Dynamic nesting support for primary serializers was introduced under #15087.

Impact

Usage of an obsolete nested serializer can be replaced by initializing the corresponding primary serializer with nested=True.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status: accepted This issue has been accepted for implementation type: deprecation Removal of existing functionality or behavior
Projects
None yet
Development

No branches or pull requests

1 participant