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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add jsonp parameter in API as a fallback to match documentation #2567

Merged
merged 2 commits into from Oct 14, 2019

Conversation

@bluemorbo
Copy link
Contributor

bluemorbo commented Oct 13, 2019

Issue #2566 was recently opened, reporting that the API documentation does not match the actual codebase. In an effort to standardise the request parameter required to support JSONP functionality, we've decided to remain backwards compatible while adding the jsonp parameter too, in order to make the API documentation correct.

If you need anything else, just shout 馃槃

@dgw
dgw approved these changes Oct 13, 2019
@dgw dgw added the api label Oct 13, 2019
@ozh ozh merged commit 80256f4 into YOURLS:master Oct 14, 2019
2 checks passed
2 checks passed
Scrutinizer 2 new issues
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can鈥檛 perform that action at this time.