Context Navigation

← Previous Ticket
Next Ticket →

#39023 closed task (blessed) (fixed)

REST API: show_in_rest property missing from the register_post_type() and register_taxonomy() args docblocks

Reported by:	johnbillion	Owned by:	pento
Priority:	normal	Milestone:	4.7
Component:	REST API	Version:	4.7
Severity:	normal	Keywords:	has-patch commit dev-reviewed
Cc:		Focuses:	docs, rest-api

Description

[38832] (#38373) introduced a show_in_rest property for post types and taxonomies. This property is not documented in the args docblocks of the register_post_type() and register_taxonomy() functions.

Attachments (2)

39023.patch (1.9 KB ) - added by ketuchetan 10 years ago.
39023.2.patch (20.7 KB ) - added by rahulsprajapati 10 years ago.: Add args docblocks of the register_post_type() and register_taxonomy() for 'show_in_rest', 'rest_base', and 'rest_controller_class' arguments.

Download all attachments as: .zip

Change History (12)

#1 @ketuchetan
10 years ago

Keywords has-patch added; needs-patch removed

Hi @johnbillion

I have added my patch can you please look into it and please let me know if I understand the ticket correctly or not.

If I am wrong then please can you please tell me the more description about this ticket please.

Thanks,
Chetan

@ketuchetan
10 years ago

Attachment 39023.patch added

@rahulsprajapati
10 years ago

Attachment 39023.2.patch added

Add args docblocks of the register_post_type() and register_taxonomy() for 'show_in_rest', 'rest_base', and 'rest_controller_class' arguments.

#2 @rahulsprajapati
10 years ago

@johnbillion
There are 'rest_base', and 'rest_controller_class' arguments also missing from arg docblock.
I've added in 39023.2.

#3 @rahulsprajapati
10 years ago

Keywords needs-docs removed

#4 @pento
10 years ago

Keywords dev-feedback commit added

This looks good to me. @DrewAPicture, could you confirm that you're cool with the whitespace changes?

#5 @johnbillion
10 years ago

Summary show_in_rest property missing from the register_post_type() and register_taxonomy() args docblocks → REST API: show_in_rest property missing from the register_post_type() and register_taxonomy() args docblocks

This ticket was mentioned in Slack in #core by chetansatasiya. View the logs.

10 years ago

#7 @nacin
10 years ago

Keywords dev-reviewed added; dev-feedback removed

#8 @pento
10 years ago

Owner set to pento
Status new → assigned

#9 @pento
10 years ago

Resolution → fixed
Status assigned → closed

In 39462:

Docs: Add missing REST API-related args to register_post_type() and register_taxonomy().

These functions now accept the show_in_rest, rest_base, and rest_controller_class args.

Props ketuchetan, rahulsprajapati.
Fixes #39023.

#10 @pento
10 years ago

In 39463:

Docs: Add missing REST API-related args to register_post_type() and register_taxonomy().

These functions now accept the show_in_rest, rest_base, and rest_controller_class args.

Merges [39462] to the 4.7 branch.

Props ketuchetan, rahulsprajapati.
Fixes #39023.

Note: See TracTickets for help on using tickets.

Download in other formats:

Make WordPress Core

Context Navigation

#39023 closed task (blessed) (fixed)

REST API: show_in_rest property missing from the register_post_type() and register_taxonomy() args docblocks

Description

Attachments (2)

Change History (12)

#1 @ketuchetan 10 years ago

@ketuchetan 10 years ago

@rahulsprajapati 10 years ago

#2 @rahulsprajapati 10 years ago

#3 @rahulsprajapati 10 years ago

#4 @pento 10 years ago

#5 @johnbillion 10 years ago

This ticket was mentioned in ​Slack in #core by chetansatasiya. ​View the logs. 10 years ago

#7 @nacin 10 years ago

#8 @pento 10 years ago

#9 @pento 10 years ago

#10 @pento 10 years ago