Skip to content

Fix #358: Home.component agora usa variavel atribuida ao inves do i18n#377

Merged
lguima merged 3 commits into
okfn-brasil:mainfrom
GuilhermePascon:fix/issue-358
Apr 18, 2026
Merged

Fix #358: Home.component agora usa variavel atribuida ao inves do i18n#377
lguima merged 3 commits into
okfn-brasil:mainfrom
GuilhermePascon:fix/issue-358

Conversation

@GuilhermePascon
Copy link
Copy Markdown

Português (BR) | English (US)

Comunidade

Tipo de alteração

  • 🐞 Correção de problema
  • ✨ Melhoria ou nova funcionalidade
  • 📰 Nova postagem no blog

Issues relacionadas

Issues que são relacionadas a esta Pull Request.

Resolve #358

Validação

  • Validei a alteração no link gerado pelo bot da Netlify (Deploy Preview/Preview on mobile)
  • Validei o Layout responsivo (desktop/mobile) após a implementação
  • Verifiquei o registro do deploy (Latest deploy log) e nenhum novo alerta ou erro foi adicionado

Evidências

Anexe evidências do antes e do depois da alteração (quando necessário).

Antes:
image
Depois:
image

Documentação

  • A documentação deste repositório foi atualizada (quando necessário).
  • Esta alteração requer que a documentação externa seja atualizada.

@netlify
Copy link
Copy Markdown

netlify Bot commented Nov 27, 2025

Deploy Preview for querido-diario-plataforma ready!

Name Link
🔨 Latest commit 0d91301
🔍 Latest deploy log https://app.netlify.com/projects/querido-diario-plataforma/deploys/69e2794f3bff7d00088edbe5
😎 Deploy Preview https://deploy-preview-377--querido-diario-plataforma.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@MoisesMsa
Copy link
Copy Markdown
Collaborator

Opa, Guilherme, tudo bom? Estou revisando a PR achei boa a sacada, mas precisamos pensar em um ajuste para que a tradução dessa parte da página continue funcionando também junto com a contagem. Qualquer dúvida chama lá no discord.

Copy link
Copy Markdown
Collaborator

@MoisesMsa MoisesMsa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

precisamos ajustar e manter a tradução

@lguima lguima requested a review from MoisesMsa April 17, 2026 18:20
@lguima
Copy link
Copy Markdown
Collaborator

lguima commented Apr 17, 2026

@MoisesMsa ajustei o código pra considerar tanto a tradução, quanto o dado da API

e atualizei o total de diários na tradução inglês

cc: @GuilhermePascon @hsvab

Copy link
Copy Markdown
Collaborator

@MoisesMsa MoisesMsa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Aprovado

@lguima lguima merged commit b60d8cf into okfn-brasil:main Apr 18, 2026
4 checks passed
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.

Retomar com o contador de cidades automático

4 participants