-
-
Notifications
You must be signed in to change notification settings - Fork 88
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
[V2] Adiciona suporte a múltiplas línguas usando Jekyll Polyglot #267
Conversation
❌ Deploy Preview for diciotech failed.
|
@george-gca eu finalmente consegui olhar sua PR! Eu gostei bastante, acho que definitivamente vai facilitar o projeto e tendo uma documentação boa, não vejo o motivo de não fluir bem! Vou dar um check com o pessoal nas duas PRs maiores que estamos abertas para seguirmos por esse caminho! |
@george-gca consegue dar uma avaliada nessa ideia de issue #270? Acha que faz sentido? Como issa impactaria sua implementação? |
Beleza, então vou dar continuidade na implementação. É bom que tu vais ser o teste de documentação, conseguir reproduzir o ambiente só lendo haha o/ |
Perfeito @george-gca hahahaha |
@levxyca tô dando um trato nisso agora. Por favor, não aceita NENHUMA outra PR até eu terminar isso, que tá dando um trabalho fazer o merge porque eu mudei muita coisa na estrutura. |
@levxyca chegou o momento de testar. Clona o meu branch, dá uma lida no CONTRIBUTING.md e vê se entende todo o processo, se consegue replicar, etc. Não precisa fazer a instalação manual se não quiser, ela é o caminho que eu uso pra executar localmente então essa eu garanto que funciona. O devcontainer que eu não tenho muita experiência e seria legal validar, até porque é a forma mais fácil de ter todas as dependências necessárias pra rodar o código localmente de uma forma isolada. Vê se a documentação tá clara, as explicações fazem sentido, se alguém que nunca viu Jekyll consegue pegar e começar daí. Qualquer melhoria que conseguir ver, só falar o/ PS: essas 2 ações tão falhando faz um tempo mas elas tão bem estranhas. Elas tão falhando na hora inclusive de clonar o repositório, o que é estranho. Tem até um bug aberto sobre isso, comentei lá, bora ver se alguém dá algumas resposta, mas na pior das hipóteses a gente comenta essas ações. |
@george-gca vi depois de aceitar uma PR 😭 mas fechado! vou tentar ir replicando o processo e logo mais volto aqui com os feedbacks! |
@george-gca vou começar aqui os meus feedbacks 😛 Falando de documentação no geral:
Sobre a instalação do ambiente para contribuir usando o DevContainer:
Obs:. Ainda não terminei de olhar tudo mas já vou ir deixando conforme eu for olhando os comentários aqui pra não perder. |
@levxyca boaaaaaa. Quanto as considerações:
Aí eu tentei adicionar esse texto depois de alguma forma, mas concordo que provavelmente muita gente não vai ler muito pra baixo. Acha que deixo uma explicação grande mesmo nessa parte, ou tento só deixar explícito sobre o que deve mudar? Tipo:
Quanto à diferença do |
|
Acabei de testar, e dá também pra abrir o repositório nessa nova versão no GitHub Codespaces e tem um limite gratuito que dá pra usar. Só abrir e ele já abre no esquema, com a página carregada pra visualizar (graças ao devcontainer), sem precisar instalar ou configurar mais nada. |
@george-gca ah, como eu amo a tecnologia, o GitHub Codespaces facilita tudo HAHAHHA alguns cliques e: |
Mais fácil que isso pra configurar o ambiente de desenvolvimento não dá pra ficar haha. |
@george-gca SIM, tá muito bom, sério! Única coisa que tá me faltando agora é sobre o Docker que ainda não consegui tempo pra fazer o passo a passo certinho e usar o Devcontainer. |
@george-gca só pra te avisar que estou fazendo o processo do DevContainer hoje, até o fim do dia te mando meus feedbacks e se funcionou certinho pra gente conseguir andar 🚀 |
@brunodmartins consegue dar um review aqui também? 🤩 |
@george-gca eu tive um erro na hora da conexão com o DevContainer, apesar disso a aplicação rodou normalmente 🥳🙌🏻, vou deixar o erro aqui no print: Agora, depois de seguir o passo a passo, segue a minha sugestão de modificação para essa parte da documentação:
|
Signed-off-by: George Araújo <george.gcac@gmail.com>
Signed-off-by: George Araújo <george.gcac@gmail.com>
Signed-off-by: George Araújo <george.gcac@gmail.com>
Signed-off-by: George Araújo <george.gcac@gmail.com>
Signed-off-by: George Araújo <george.gcac@gmail.com>
Signed-off-by: George Araújo <george.gcac@gmail.com>
Signed-off-by: George Araújo <george.gcac@gmail.com>
Signed-off-by: George Araújo <george.gcac@gmail.com>
Signed-off-by: George Araújo <george.gcac@gmail.com>
Signed-off-by: George Araújo <george.gcac@gmail.com>
Signed-off-by: George Araújo <george.gcac@gmail.com>
Signed-off-by: George Araújo <george.gcac@gmail.com>
Mudanças necessárias no Netlify:
Mudanças legais pra fazer no repositório:
|
Signed-off-by: George Araújo <george.gcac@gmail.com>
Signed-off-by: George Araújo <george.gcac@gmail.com>
Signed-off-by: George Araújo <george.gcac@gmail.com>
Signed-off-by: George Araújo <george.gcac@gmail.com>
Issues que eu vou criar quando a PR for aceita:
|
Signed-off-by: George Araújo <george.gcac@gmail.com>
Dica: quando for aceitar a PR é uma boa dar squash pra não virem 35 mil commits diferentes pra main. |
Pessoal, voltei de recesso 🫣😂 E finalmente temos o mergeeeeeeeee 🚀 @george-gca acredito que no geral tá tudo redondinho 👀 |
Descrição de PR
Essa PR é uma continuação da #217, mas preferi criar uma nova PR porque era mais fácil fazer do 0 a partir da main do que fazer um merge naquela branch. Inclusive tenta priorizar analisar essa PR @levxyca antes de aprovar as outras. Acho que é mais fácil depois a gente pegar a contribuição da galera de termos e só aplicar aqui, ou pedir pra refazerem a partir da nova main.
Explicando um pouco mais o jekyll: por definição ele é um gerador de sites estáticos. Pensa nele como um "compilador de sites", ele faz um build usando vários arquivos e gera os arquivos html, css e js finais. Por isso que com ele dá pra fazer essas coisas.
Principais novidades:
_data/LANG/LETRA.yml
_assets/LANG/cards.json
e unificados em um json por língua (resolve Implementação de Tradução nos Cards #185), sendo que eles não precisam ser modificados e os arquivos json finais não precisam ser versionados (são gerados durante o build do jekyll)style.css
é gerado durante o build do jekyll, ou seja, só existe os arquivos scss no git então a modificação precisa ser feita nelesFaltam:
documentar tudo isso direitinho (farei assim que aprovada a ideia, ainda nesta PR)traduzir todos os termos do dicionárioPra ter uma noção, basicamente a estrutura de pastas do projeto vira isso (tirei os arquivos que não serão modificados da visualização, como README, etc, pra simplificar):
E esse é o resultado do build do jekyll, ou seja, as pastas que vão subir pro Netlify:
Pra acessar o site em português é só acessar normal, e o em inglês é só acessar com
/en-us
.Issue relacionado
Motivações
Pra não copiar todo o texto de novo, as motivações e etc estão na #217.
Informações adicionais
O build do Netlify vai falhar até mudar pra usar o Jekyll nele, então gerei uma versão no Netlify do meu branch aqui.