wpbrasil / odin

Framework and base theme for development with WordPress.
http://wpod.in
565 stars 195 forks source link

Add TGM Plugin Activation #448

Closed allysonsouza closed 7 years ago

allysonsouza commented 7 years ago

Add TGM Plugin Activation resources to theme, making possible to require or recommend plugins to be installed with the theme.

allysonsouza commented 7 years ago

https://github.com/wpbrasil/odin/commit/4e87c324abbff4a97995e1a9628dbc0fb15c2d94

claudiosanches commented 7 years ago

Sinceramente eu não gosto disso, principalmente por enfiar um projeto inteiro goela abaixo de todo mundo, sem contar que agora incluído aqui dentro, qualquer commit que eles fizerem, vamos ter que manualmente incluir aqui também. Seria um pouco melhor ter usado Git Submodule ou Composer para manter uma copia do TGM, mas copiar e colar ele aqui é simplesmente horrível demais.

allysonsouza commented 7 years ago

Vou estudar como fazer então, apenas fiz do modo que sabia no momento. Concordo que com submodule ou composer fica melhor, sem dúvidas. Vou adicionar uma taks para melhor isso nos To Do's do projeto.

claudiosanches commented 7 years ago

Sugerir ou dar um exemplo de como usar, ou incluir como opção, acho que seria da hora. Mas se vai usar realmente o TGM, então seria legal manter um sync com algo que indica qual versão esta usando.

allysonsouza commented 7 years ago

Acho que é legal ter ele já no projeto, só tentar corrigir essa questão do sincronismo. Falando com o @adammacias ele tem notado também como é uma necessidade recorrente em alguns projetos que utilizam ACF, CMB2, Kirki, etc., e também pela remoção de recursos do tema que entram no plugin territory, deve ter visto que criamos o repositório Odin Toolkit para começar a mover isso, assim se alguém irá utilizar recursos do plugin territory dentro do tema, pode colocar ele como requisito via o TGM.

claudiosanches commented 7 years ago

Sim eu vi, estou para arrumar umas coisas lá xD Concordo que TGM tem seu valor, apenas dizendo que não é a única opção ou que é valido para todos os projetos.

allysonsouza commented 7 years ago

Removi o TGM inserido diretamente no projeto e adicionei um breve composer.json para isso. Apenas mantive o arquivo required-plugins.php que é específico do tema, assim quem for utilizar TGM é só dar um composer install e descomentar no functions o require do required-plugins.php

https://github.com/wpbrasil/odin/commit/09c85cb45f9a6f756cf134dea02f033f7ef0dcb7

adammacias commented 7 years ago

@allysonsouza como faz para o TGM carregar a tradução PT_BR nativa do plugin?

Por favor, crie um checklist na descrição desta issue para corrigir essa questão da tradução, e também para criarmos a documentação de como fazer o uso desta feature com composer install e etc.

Porque tirou o vendor do .gitignore? 😮

allysonsouza commented 7 years ago

@adammacias gostaria de saber também, talvez seja algum problema do TGM.

Checklist criado, e vendor será ignorado propriamente, tirei por cabaçada :)

claudiosanches commented 7 years ago

Alias, bom que tem o composer assim para instalar outras coisas no projeto.