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: clarify server action behavior with middleware #74458

Closed
wants to merge 1 commit into from

Conversation

terry422
Copy link

@terry422 terry422 commented Jan 1, 2025

I added a line to the "Server Actions and Mutations" page to clarify that calling Server Actions triggers middleware execution as well.

@ijjk ijjk added the Documentation Related to Next.js' official documentation. label Jan 1, 2025
@ijjk
Copy link
Member

ijjk commented Jan 1, 2025

Allow CI Workflow Run

  • approve CI run for commit: 8b49d05

Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer

@terry422
Copy link
Author

terry422 commented Jan 5, 2025

I just found out that not all calls to Server Actions result in a POST request (e.g., calling it inside a Server Component). It could be confusing without adding that middlewares run only when such a request is made.

I am closing this PR to propose a revised version.

@terry422 terry422 closed this Jan 5, 2025
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 19, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Documentation Related to Next.js' official documentation. locked
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants