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

Declare compatibility with pulpcore 3.70 #791

Merged
merged 1 commit into from
Feb 5, 2025
Merged

Conversation

gerrod3
Copy link
Contributor

@gerrod3 gerrod3 commented Feb 4, 2025

fixes: #784

@gerrod3 gerrod3 force-pushed the 370-compat branch 5 times, most recently from 82d2790 to ce2cea5 Compare February 4, 2025 22:53


@extend_schema_field(OpenApiTypes.OBJECT)
class JSONObjectField(serializers.JSONField):
Copy link
Contributor

Choose a reason for hiding this comment

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

The reason this can be removed, I presume, because tfranzel/drf-spectacular#1095 was fixed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

See my messages in pulp-dev (https://matrix.to/#/!aVApiNMtnstWbwDcVU:matrix.org/$GRdEDvdyB04MXXbc66NWVNhCaI_VgdnhUb2xuJ1OJWs?via=matrix.org&via=ctrl-c.liu.se&via=x9c4.spdns.org), but the tldr is that with 3.70 we are using a new version of the bindings generator. This new version introduces pydantic models to check the types, so the mismatch between what we say the field is in the api and what we accept causes errors in the new bindings. This previous change was because the ruby-bindings broke when drf-spectular updated, so I have no idea if the ruby bindings will break again when removing this, but they should also be using the new bindings generator so hopefully they don't.

Copy link
Contributor

Choose a reason for hiding this comment

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

Nice!

@gerrod3 gerrod3 force-pushed the 370-compat branch 3 times, most recently from ce4f29b to 1f4b07c Compare February 4, 2025 23:20
@gerrod3 gerrod3 merged commit f622665 into pulp:main Feb 5, 2025
12 checks passed
@gerrod3 gerrod3 deleted the 370-compat branch February 5, 2025 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[core 3.70 compat] Django storages config deprecation
2 participants