Skip to content

post_type url parameter - #11

Open
hiro288 wants to merge 1 commit into
rmorse:masterfrom
hiro288:master
Open

post_type url parameter#11
hiro288 wants to merge 1 commit into
rmorse:masterfrom
hiro288:master

Conversation

@hiro288

@hiro288 hiro288 commented Jan 15, 2015

Copy link
Copy Markdown

fixed post_type url parameter

fixed post_type url parameter
@rmorse

rmorse commented Jan 15, 2015

Copy link
Copy Markdown
Owner

Hey @naeemnur I'm afraid this is incorrect. The post_type parameter is native WordPress - so will work anywhere - however it only takes one argument. I implemented post_types so that we can pass multiple post types to it - eg post_types=post,page :)

@rmorse rmorse closed this Jan 15, 2015
@hiro288

hiro288 commented Jan 15, 2015

Copy link
Copy Markdown
Author

but it doesnt seems to work when you set 1 cpt

@rmorse

rmorse commented Jan 15, 2015

Copy link
Copy Markdown
Owner

Hmm I don't recall this being the case but I'll take a look - the change you have made above does effectively remove support for multiple post types - so the solution would be to fix post_types to also work with a single post type rather than switching it all over to use post_type

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants