-
Notifications
You must be signed in to change notification settings - Fork 401
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
refactor(site description): change the description to be consistent w… #1498
Conversation
…ith the TabNews purpose.
Someone is attempting to deploy a commit to the TabNews Team on Vercel. A member of the Team first needs to authorize it. |
Essa PR tá pronta pro merge? É algo tão simples, de fácil implementação em produção. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Me parece que faltou mudar aqui:
tabnews.com.br/pages/interface/components/Head/index.js
Lines 17 to 19 in 086931b
title: 'TabNews: Conteúdos para quem trabalha com Programação e Tecnologia', | |
image: `${webserverHost}/default-image-share.png`, | |
description: 'Conteúdos com valor concreto para quem trabalha com tecnologia.', |
Quando for mergear, vale a pena mudar o About
do repositório também.
Além disso, é importante ler o comentário #1497 (comment) para pensarmos numa descrição melhor.
Pessoal, estou fechando este PR para organizar o repositório porque já está aberto há algum tempo e não teve um andamento conforme o comentário do Felipe em #1497 (comment). O issue #1497 ainda está aberto para implementação, mas é preciso chegar num consenso antes sobre qual seria um bom texto. Quando surgir uma boa descrição, um novo PR pode ser aberto, criando uma nova branch a partir do Obrigado pelo PR e pelo issue, @VictorManhani! |
PR sobre a issue #1497