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

[Pages] Add clarification to call order of pages onRequest methods #19290

Open
tjsr opened this issue Jan 17, 2025 · 0 comments
Open

[Pages] Add clarification to call order of pages onRequest methods #19290

tjsr opened this issue Jan 17, 2025 · 0 comments
Assignees
@irvinebroque @GregBrimble @WalshyDev @aninibread @ToriLindsay
Labels
content:edit Request for content edits documentation Documentation edits product:pages

Comments

@tjsr
Copy link

tjsr commented Jan 17, 2025

Existing documentation URL(s)

https://developers.cloudflare.com/pages/functions/api-reference/

What changes are you suggesting?

In the list of onRequest method variants available, the documentation does not discuss what happens if both the generic and specific-verb method are defined and exported.

Some simple extra text to clarify the behaviour of functions will prevent people needing to test this behaviour themselves.

Additional information

I have created a draft PR for a proposed addition of wording at #19289
@tjsr tjsr added content:edit Request for content edits documentation Documentation edits labels Jan 17, 2025
@tjsr tjsr mentioned this issue Jan 17, 2025
Draft
1 task
@tjsr tjsr changed the title Add clarification to call order of pages onRequest methods [Pages] Add clarification to call order of pages onRequest methods Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@irvinebroque irvinebroque

@GregBrimble GregBrimble

@WalshyDev WalshyDev

@aninibread aninibread

@ToriLindsay ToriLindsay

Labels
content:edit Request for content edits documentation Documentation edits product:pages
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@irvinebroque @GregBrimble @WalshyDev @tjsr @aninibread @ToriLindsay