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.

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

Create Odin Custom Post Status Closes #262 #266

Closed
wants to merge 12 commits into from
Closed

Create Odin Custom Post Status Closes #262 #266

wants to merge 12 commits into from

Conversation

rands0n
Copy link

@rands0n rands0n commented Mar 7, 2015

Terminei a classe gente. Com umas pesquisadas a mais é claro. Mas agora ficará para todos que quiserem usar :)

A maneira de usar a classe é assim, depois teremos que criar uma documentação para a mesma:

$post_types = array('post');

$args = array(
    'applied_label' => 'Archive',
    'label' => 'Archive',
    'public' => true,
    'exclude_from_search' => false,
    'show_in_admin_all_list' => true,
    'show_in_admin_status_list' => true,
);

new Odin_Post_Status( 'archive', $post_types, $args );

Tirei algumas screenshots do teste que fiz aqui em minha máquina:

screenshot from 2015-03-07 00 57 23

oie_1jttgjggvkyl

oie_yrbnuhsr23hv

@rands0n rands0n changed the title Create Odin Custom Post Status Create Odin Custom Post Status Closes #262 Mar 7, 2015
@claudiosanches
Copy link
Member

Precisa seguir o coding standards do WordPress: https://make.wordpress.org/core/handbook/coding-standards/php/

@rands0n
Copy link
Author

rands0n commented Mar 8, 2015

Acho que está ok agora. Pode dar uma verificada?

@claudiosanches
Copy link
Member

@claudiosanches
Copy link
Member

Outra coisa, evite isso $script = <<<SCRIPT não é muito usado no WordPress.
Além que mesmo usando isso você acabou errado com </script>";.
Não teria porque ter ";.

@rands0n
Copy link
Author

rands0n commented Mar 8, 2015

Muito obrigado por deixar no modelo do WordPress 🎉 Se ainda estiver algo avise ;)

@adammacias
Copy link
Member

@randsonjs Eu tentei testar aqui, mas não apareceu nenhum status personalizado 😢 Está funcionando normal aí depois depois últimos commits?

@rands0n
Copy link
Author

rands0n commented Mar 16, 2015

Realmente @adammacias. Depois que removi o heredoc não está funcionando. Assim que tiver um tempo adiciono mais commits e aviso por aqui :)

@adammacias
Copy link
Member

Beleza, fico no aguardo #262

@matheusgimenez
Copy link
Member

Posso assumir esse PR aqui? Tô afim de recriar isso colocando mais nos padrões

@matheusgimenez
Copy link
Member

Só uma coisa: Quanto ao codigo JS no meio do PHP penso que seria interessante jogar essa lista de status num array e trazer para o JS via wp_localize_script()

@adammacias
Copy link
Member

@matheusgimenez por mim beleza, o @randsonjs não responde desde 16 de março 😢

@rands0n
Copy link
Author

rands0n commented Jul 8, 2015

Acho maneiro fazer. Pode comitar no meu repô que já aparece aqui as modificações! Nem precisa forkar pra abrir outro PR.

Toda ajuda é bem vindo 😄

@rands0n
Copy link
Author

rands0n commented Jul 8, 2015

Em alguns commits atrás a classe estava funcionando. Se puderem apenas colocar no padrões do WP fico feliz ;)

matheusgimenez added a commit to matheusgimenez/odin that referenced this pull request Jul 11, 2015
matheusgimenez added a commit to matheusgimenez/odin that referenced this pull request Jul 11, 2015
@rands0n
Copy link
Author

rands0n commented Jul 13, 2015

Fechando pull request em favor do #310!

@rands0n rands0n closed this Jul 13, 2015
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.

4 participants