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

Adiciona preview para o site #313

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Adiciona preview para o site #313

wants to merge 1 commit into from

Conversation

trevineju
Copy link
Member

@trevineju trevineju commented Oct 22, 2024

Português (BR) | English (US)

Comunidade

Tipo de alteração

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

Issues relacionadas

Não resolve a issue #303, mas adiciona uma solução temporária

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

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.

Copy link

netlify bot commented Oct 22, 2024

Deploy Preview for querido-diario-plataforma ready!

Name Link
🔨 Latest commit 3105373
🔍 Latest deploy log https://app.netlify.com/sites/querido-diario-plataforma/deploys/6717cdcce51c5b000861993d
😎 Deploy Preview https://deploy-preview-313--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 site configuration.

@trevineju
Copy link
Member Author

trevineju commented Oct 22, 2024

@lguima, vc poderia dar uma revisão aqui? o app.component.html é o lugar certo de se adicionar as tags?
como estavamos falando na issue, a ideia no momento seria uma solução só pro site todo.

testei com https://www.opengraph.xyz/url/https%3A%2F%2Fdeploy-preview-313--querido-diario-plataforma.netlify.app%2F e parece ter funcionar, mas não tenho certeza se o deploy-preview é o jeito certo de se testar isso.

também achei o uso de metaService na base de código, em search-form.component.ts que me deixou em dúvida. Não que a solução tenha que ser adicionada no arquivo search-form, mas a ideia de que o caminho certo é importar @angular/platform-browser para implementar isso: https://v17.angular.io/api/platform-browser/Meta

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant