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

Cozinhar os metadados no getStaticProps() #650

Closed
filipedeschamps opened this issue Aug 15, 2022 · 3 comments
Closed

Cozinhar os metadados no getStaticProps() #650

filipedeschamps opened this issue Aug 15, 2022 · 3 comments
Labels
front Envolve modificações no frontend

Comments

@filipedeschamps
Copy link
Owner

filipedeschamps commented Aug 15, 2022

Contexto

Dando continuidade na issue #622 que traz a implementação de @aprendendofelipe no PR #627, temos uma margem para melhorar ainda mais a abertura de uma página no TabNews, incluindo acelerar o Time to Interactive (TTI), principalmente para páginas que possuem um body muito grande, que é limpar os metadados antes de serem "cozinhados" pelo Next.js.

Execução

Hoje estamos utilizando o novo model remove-markdown.js com o fix do @aprendendofelipe no cabeçalho dos arquivos de layout, por exemplo:

const cleanBody = removeMarkdown(content.body).replace(/\s+/g, ' ');

Fora isso, fazemos ainda algumas transformações, como remover as quebras de linha e tudo isso pode ser feito dentro do getStaticProps() para nada disso precisar ser processado no client-side.

@filipedeschamps filipedeschamps added the front Envolve modificações no frontend label Aug 15, 2022
@kaique-soares
Copy link
Contributor

@filipedeschamps eu tive a mesma sensação que você relatou em relação ao frontend, no PR #660

Eu subi uma branch com a implementação dessa issue aqui, mas fiquei travado para abrir o PR porque não me senti seguro se o que eu fiz, de fato, resolve a issue. Apesar da implementação ser simples, acredito, eu me peguei checando várias vezes se onde eu modifiquei era de fato o único lugar que o component DefaultLayout tava recebendo a propriedade content.

Poderia dar uma olhada?

@filipedeschamps
Copy link
Owner Author

Sensacional @kaique-soares ! Uma sugestão: mantenha o medo e abra o PR mesmo assim! Você vai ver que vamos tentar ao máximo proteger para que aqui seja um espaço em que qualquer pessoa possa tentar contribuir. Não interessa se sua implementação está certa ou não, o que interessa é que, mesmo com medo, você vai ter coragem de abrir esse PR, esta é a parte mais importante desta sua contribuição.

O que sobra, é a parte técnica, e daí a gente conversa sobre isso lá no PR, combinado? 🤝

@filipedeschamps
Copy link
Owner Author

Fechado por #661

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
front Envolve modificações no frontend
Projects
None yet
Development

No branches or pull requests

2 participants