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

Public JSON API section: add petition content endpoints and information about CORS vs. JSONP #145

Merged
merged 5 commits into from
May 21, 2024

Conversation

lavaturtle
Copy link

This adds documentation for the new API endpoints /api/petitions/foo/what.json and /api/petitions/foo/why.json.

image

It also adds some explanatory text discussing the difference between CORS and JSONP, and identifies whether each public JSON API endpoint supports CORS, JSONP, or neither.
image

@lavaturtle lavaturtle added the merge-candidate This PR is ready to be merged, once it is reviewed and tested label May 21, 2024
Copy link

@anero anero left a comment

Choose a reason for hiding this comment

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

👍 looks good!

source/includes/json/_intro.md Outdated Show resolved Hide resolved
source/includes/json/_intro.md Outdated Show resolved Hide resolved
@lavaturtle lavaturtle merged commit c915a78 into mf-cs-documentation May 21, 2024
2 checks passed
@lavaturtle lavaturtle deleted the petition-json-cors branch May 21, 2024 20:43
lavaturtle added a commit that referenced this pull request May 21, 2024
…information about CORS vs. JSONP (#145)

generated from commit c915a78
anero added a commit that referenced this pull request May 28, 2024
…information about CORS vs. JSONP (#145)

generated from commit c915a78
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge-candidate This PR is ready to be merged, once it is reviewed and tested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants