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.

Sign up for GitHub

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

Jump to bottom

Separate cache for HTTP methods #2720

Merged
merged 1 commit into from
Jan 10, 2025
Merged

Separate cache for HTTP methods #2720

merged 1 commit into from
Jan 10, 2025

Conversation

gregberge
Copy link
Contributor

@gregberge gregberge commented Jan 10, 2025 •
edited
Loading

Fix GITBOOK-OPEN-1WZE
Fix GITBOOK-OPEN-1WZF

Server actions use the same endpoint with a "POST" method. In Cloudflare, we serve the cache from the GET request.

In that PR I do two things:

@changeset-bot changeset-bot
Copy link

changeset-bot bot commented Jan 10, 2025 •
edited
Loading

🦋 Changeset detected

Latest commit: 3805322

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
gitbook Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@gregberge
Separate cache for http methods
3805322 
Fix GITBOOK-OPEN-1WZE
Fix GITBOOK-OPEN-1WZF
@gregberge gregberge force-pushed the separate-cash-for-post branch from 38ab099 to 3805322 Compare January 10, 2025 14:43
@gregberge gregberge changed the title Separate cache for post methods Separate cache for HTTP methods Jan 10, 2025
@gregberge gregberge requested a review from emmerich January 10, 2025 14:44
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jan 10, 2025 •
edited
Loading

GitBook Preview
Latest commit: https://329e3f7f.gitbook-open.pages.dev
PR: https://pr2720.gitbook-open.pages.dev

@argos-ci Argos CI
Copy link

argos-ci bot commented Jan 10, 2025 •
edited
Loading

The latest updates on your projects. Learn more about Argos notifications ↗︎

Build Status Details Updated (UTC)
default (Inspect) 👍 Changes approved 26 changed Jan 10, 2025, 2:52 PM

@gregberge gregberge merged commit a6f6591 into main Jan 10, 2025
9 checks passed
@gregberge gregberge deleted the separate-cash-for-post branch January 10, 2025 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@clairechabas clairechabas clairechabas approved these changes

@emmerich emmerich Awaiting requested review from emmerich

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gregberge @clairechabas