"all" not supported as Payload `method` type though in docs

[cgilly2fast]

Describe the Bug

The payload endpoint type definition notes being able to set an endpoint to "all"

but its not actually possible

Can "all" be added?

/**
 * Docs: https://payloadcms.com/docs/rest-api/overview#custom-endpoints
 */
export type Endpoint = {
  /** Extension point to add your custom data. */
  custom?: Record<string, any>

  /**
   * Middleware that will be called when the path/method matches
   *
   * Compatible with Web Request/Response Model
   */
  handler: PayloadHandler
  /** HTTP method (or "all") */
  method: 'connect' | 'delete' | 'get' | 'head' | 'options' | 'patch' | 'post' | 'put'
  /**
   * Pattern that should match the path of the incoming request
   *
   * Compatible with the Express router
   */
  path: string
  /**
   * Please add "root" routes under the /api folder in the Payload Project.
   * https://nextjs.org/docs/app/api-reference/file-conventions/route
   *
   * @deprecated in 3.0
   */
  root?: never
}

Link to the code that reproduces this issue

https://github.com/payloadcms/payload/blob/main/packages/payload/src/config/types.ts

Reproduction Steps

Create and endpoint, and attempt to use "all"

Which area(s) are affected? (Select all that apply)

area: core

Environment Info

`3.64.0`

[paulpopus]

"all" is not a valid HTTP method as part of the web API spec so it looks like this comment was leftover from Expressjs.

The pattern you could use is a reusable handler or creating a utility function if you did want to run the same code on all methods, though generally speaking we recommend being explicit.

Added a PR to remove this comment

[github-actions]

🚀 This is included in version v3.66.0