Opened 8 years ago
Closed 8 years ago
#35799 closed enhancement (fixed)
Update class header for WP_REST_Request to communicate parameter priorities
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 4.5 | Priority: | normal |
Severity: | normal | Version: | |
Component: | REST API | Keywords: | needs-docs |
Focuses: | docs | Cc: |
Description
We should summarize https://github.com/WP-API/WP-API/issues/2114#issuecomment-182873506 in the class header so endpoint developers know what they're getting into.
Attachments (2)
Change History (8)
This ticket was mentioned in Slack in #core by chriscct7. View the logs.
8 years ago
This ticket was mentioned in Slack in #core-restapi by danielbachhuber. View the logs.
8 years ago
#4
@
8 years ago
- Owner set to DrewAPicture
- Status changed from new to reviewing
@danielbachhuber Based on the comment on GitHub, I've adjusted your patch a little bit to try to be more clear about what's expected. Would you mind reviewing 35799.diff for accuracy?
Note: See
TracTickets for help on using
tickets.
@danielbachhuber You're proposing a docbloc that summarizes that comment correct? Or like a codex doc