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.

Sign up for GitHub

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

Jump to bottom

Closes #17143: Deprecate the use of dedicated nested API serializers #17164

Merged

jeremystretch merged 1 commit into feature from 17143-deprecate-nested-serializers

Aug 15, 2024

Member

jeremystretch commented Aug 14, 2024 •

edited

Loading

Closes: #17143

Relocate all nested serializer classes that are still required (i.e. to avoid circular imports) to a new nested module within api/serializers_/ under each app.
Remove all imports of the nested_serializers modules.
Raise a DeprecationWarning when importing any of the now-obsolete nested_serializers modules. These will be completely removed in NetBox v4.2 per Remove all obsolete nested REST API serializers #17165.


          Closes #17143: Deprecate the use of dedicated nested API serializers

646244f

jeremystretch requested a review from arthanson

August 14, 2024 19:29

jeremystretch added the beta label

arthanson approved these changes

View reviewed changes

jeremystretch merged commit e3d681b into feature

6 checks passed

jeremystretch deleted the 17143-deprecate-nested-serializers branch

August 15, 2024 12:00

github-actions bot locked as resolved and limited conversation to collaborators

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

beta