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

docs: Serialization API uses POST #44439

Merged
merged 2 commits into from
Jun 19, 2024
Merged

Conversation

alexyarosh
Copy link
Contributor

Call more attention to the fact that export API uses POST, not GET, because this keeps coming up. Also added explanation (from this slack thread) why we made this decision so it wouldn't seem like an arbitrary constraint.

@alexyarosh alexyarosh added Type:Documentation backport Automatically create PR on current release branch on merge labels Jun 19, 2024
@alexyarosh alexyarosh self-assigned this Jun 19, 2024
Copy link
Member

@jeff-bruemmer jeff-bruemmer left a comment

Choose a reason for hiding this comment

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

Thanks, @alexyarosh.

@alexyarosh alexyarosh merged commit 301ab32 into master Jun 19, 2024
119 of 131 checks passed
@alexyarosh alexyarosh deleted the docs-ay-serialization-api-post branch June 19, 2024 20:24
github-automation-metabase pushed a commit that referenced this pull request Jun 19, 2024
* call attention to POST

* wording
github-automation-metabase added a commit that referenced this pull request Jun 19, 2024
* call attention to POST

* wording

Co-authored-by: Alex Yarosh <alexandra@metabase.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport Automatically create PR on current release branch on merge Type:Documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants