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

add setting for removing Browsable API from v2 endpoints [#186670646] #635

Merged
merged 1 commit into from
Dec 13, 2023

Conversation

uraniumanchor
Copy link
Collaborator

Contributing to the Donation Tracker

  • I've added tests or modified existing tests for the change.
  • I've humanly end-to-end tested the change by running an instance of the tracker.

Issue from Pivotal Tracker

https://www.pivotaltracker.com/story/show/186670646

Description of the Change

This was hardcoded in the live fixes branch but it probably deserves a more formal setting.

Verification Process

Flipped the DEBUG or the new explicit flag on and off and made sure the browsable API either worked or didn't.

@uraniumanchor uraniumanchor added this pull request to the merge queue Dec 13, 2023
Merged via the queue into master with commit dc1d201 Dec 13, 2023
15 checks passed
@uraniumanchor uraniumanchor deleted the 186670646-disable-browseable-api branch December 13, 2023 04:57
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