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

Este commit insere uma Theme Options #87

Closed
wants to merge 1 commit into from
Closed

Este commit insere uma Theme Options #87

wants to merge 1 commit into from

Conversation

joelgarciajr84
Copy link

Issue #77

@valeriosouza
Copy link
Contributor

@joelgarciajr84 ou eu, ou alguem vai dar uma revisa e dar um merge

@nicholasio
Copy link
Contributor

@joelgarciajr84 eu nem dei fetch, mas ao olhar o que você fez, acho que você não compreendeu ou não acompanhou a discussão. Nós vamos fazer as opções do tema com Customizer, já tinha ficado decidido e recentemente o theme review team do wordpress.org criou uma regra que torna obrigatório a utilização do theme customizer para esse fim. Eles não permitem mais esse tipo de coisa que você fez, eles querem tudo no customizer.

Infelizmente não dá pra dar merge ;(

@nicholasio nicholasio closed this Jun 19, 2015
@joelgarciajr84
Copy link
Author

Hum entendi, vou ver como exatamente precisa ser feito, e se ainda em tempo, altero. Realmente não havia acompanhado a discussão, não sabia que era obrigatório o uso do customizer

@joelgarciajr84
Copy link
Author

Caso ninguém faça até a próxima madrugada, enviarei a nova versão usando o Customizer.

@leobaiano
Copy link
Contributor

Além disso, @joelgarciajr84, tenta acompanhar o que já está feito, metade do código que você escreveu já havia sido criado no incs/customizer.php.

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