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

Enable django pretty api format #1891

Merged

Conversation

jerabekjiri
Copy link
Contributor

Issue: AAH-2148

This is for oci_env only. We should enable this for beta-galaxy as well.

@himdel
Copy link
Collaborator

himdel commented Sep 15, 2023

Hm.. is it just me or is switching from/to the raw view broken on some screens?

Works:

http://localhost:8002/api/automation-hub/pulp/api/v3/status/?format=api
20230915200136

http://localhost:8002/api/automation-hub/pulp/api/v3/status/?format=json
20230915200056

Breaks:

http://localhost:8002/api/automation-hub/pulp/api/v3/repositories/?format=api
20230915200125

http://localhost:8002/api/automation-hub/pulp/api/v3/repositories/?format=json
20230915200117

I'm guessing there's a global pulp filter that's disallowing unknown params and is missing an exception for ?format when PULP_DEBUG is on?

(Otherwise LGTM :))

@jerabekjiri
Copy link
Contributor Author

jerabekjiri commented Sep 18, 2023

@himdel That would make sense. 👍 I'll look into it, and if it proves to be true, I'll create an issue on the Pulp side.

@jerabekjiri jerabekjiri merged commit ba1d248 into ansible:master Sep 18, 2023
21 checks passed
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.

3 participants