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

meta tags #38

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

LeonardoRafaelDev
Copy link

Salve! 💜📝

  1. Implementação do Open Graph: 🚀

    • Adicionado o título, descrição, link e imagem do Open Graph, melhorando o compartilhamento do nosso conteúdo em plataformas como Facebook e LinkedIn.
  2. Aprimoramento das Metatags do Twitter: 🐦

    • Implementado o suporte às metatags do Twitter, incluindo nome, título, descrição, link, imagem, texto alternativo da imagem e autor. Agora, o compartilhamento do nosso conteúdo no Twitter será mais informativo e atraente.
  3. Correção da Rota do Favicon: 🖼️

    • Corrigido o problema que impedia a exibição correta do favicon do site, garantindo uma melhor identificação visual em navegadores e favoritos.
  4. Atualização das Versões do Package.json: 📦

    • Atualizados as versões do React, Node e Serve no arquivo package.json para garantir que estejamos utilizando as versões mais recentes e estáveis das dependências.
  5. Adição da Configuração Tailwind ao remix.config: 🎨

    • Adicionada a configuração tailwind:true ao remix.config para habilitar o uso do Tailwind CSS em nosso projeto, possibilitando um desenvolvimento mais rápido e consistente da interface.

fiz as alteracoes no package.json, tailwind e remix por conta que estava tendo problemas na hora de rodar o projeto, mas o @JVictorV me orientou direitinho pra fazer isso.
por algum motivo nao consegui testar as meta tags localmente no metatags.io mas creio que esteja tudo certo

ps: irei fazer outros pr no projeto para corrigir alguns erros visuais que vi na landing.

@vercel
Copy link

vercel bot commented Jul 28, 2023

@LeonardoRafaelDev is attempting to deploy a commit to the he4rt-ssr Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Contributor

@JVictorV JVictorV left a comment

Choose a reason for hiding this comment

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

Ficou chique, so algumas coisinhas pra arrumar

package-lock.json Outdated Show resolved Hide resolved
public/images/banner-meta.png Outdated Show resolved Hide resolved
tsconfig.json Outdated Show resolved Hide resolved
app/root.tsx Outdated Show resolved Hide resolved
@LeonardoRafaelDev
Copy link
Author

LeonardoRafaelDev commented Aug 1, 2023

Ficou chique, so algumas coisinhas pra arrumar

  • alteracoes feitas @JVictorV ! obrigado pelo feedback

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.

3 participants