feat(next-link): added custom Link to Header

[aprendendofelipe]

Baseado em #682, só que utilizando o componente Link customizado

[vercel]

@luantoningalvan is attempting to deploy a commit to the TabNews Team on Vercel.

To accomplish this, @luantoningalvan needs to request access to the Team.

Afterwards, an owner of the Team is required to accept their membership request.

If you're already a member of the respective Vercel Team, make sure that your Personal Vercel Account is connected to your GitHub account.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated
tabnews	✅ Ready (Inspect)	Visit Preview	Aug 26, 2022 at 3:45PM (UTC)

[aprendendofelipe]

O problema de usar o mesmo componente Link customizado para o Header é que acabava ficando com duas tags <a> aninhadas (a do PrimerLink e a do Header.Link), então acabei criando o componente HeaderLink customizado.

Agora que vi a velocidade de navegação desses links do Header com o next/link, não consigo mais imaginar ficar sem... hahaha... Então quanto mais alternativas para isso ir logo para produção, melhor 🚀

[filipedeschamps]

Merged!!! Let's gooooooo!!! Parabéns @luantoningalvan e @aprendendofelipe por esse upgrade de UX 🤝

E por curiosidade, esse foi o diff que encontrei (Antes/Depois), só mudou a ordem de duas propriedades.

[luantoningalvan]

@filipedeschamps cara, deu ruim então, pois os ajustes no menu de usuário não foram aplicados

[filipedeschamps]

Na verdade eu não peguei o diff da página inteira, só do cabeçalho (que era minha maior preocupação).

Mas vendo aqui, de fato o prefetch não está acontecendo para os itens no menu... hmm

[edit]

No menu do usuário eu digo

[luantoningalvan]

Talvez foi na transposição do código, já que tudo já está fechado e mergeado eu posso fazer uma branch nova baseada na main e fazer o commit desse resto, pode ser?

[aprendendofelipe]

Interessante. Na primeira versão que eu subi está funcionando:

https://tabnews-j94rhjkcy-tabnews.vercel.app/publicar

E, teoricamente, entre as duas versões eu só mudei os links do Header e não do menu, mas mudei a forma de importar o Link, então deve estar retornando undefined e seria o mesmo que não informar o as.

Aproveitando, vou tirar o next/link do Editar perfil, pois está fazendo request atoa

[luantoningalvan]

Sei lá o que foi então ksksks, @aprendendofelipe faz a boa aí de deixar o código master no jeito?

[aprendendofelipe]

Era isso mesmo, a forma de importar o Link que estava retornando undefined

Testem agora

https://tabnews-git-next-link-user-menu-tabnews.vercel.app/