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

Perfil: Adicionar links permanentes para conteúdo gerado pelo usuário #1693

Closed
wants to merge 1 commit into from

Conversation

mthmcalixto
Copy link
Contributor

@mthmcalixto mthmcalixto commented May 13, 2024

Mudanças realizadas

Implementar links permanentes para conteúdo gerado pelo usuário.

Limite de 5 mudanças de nome. (no futuro, adicionar de acordo com cargo)
Links de contéudos e comentários são permanentes, redireciona para o novo dono, por meio da url.
O perfil antigo é removido, não terá redirecionamento.

Novo aviso;
image

Primeiro conteúdo antes da troca;
image

Primeiro conteúdo apos a troca;
image

Acessando user antigo;
image

Limite de 5;
image

Resolve #1665

Tipo de mudança

[x] Nova funcionalidade

Checklist:

  • As modificações não geram novos logs de erro ou aviso (warning).
  • Eu adicionei testes que provam que a correção ou novo recurso funciona conforme esperado.
  • Os antigos estão passando localmente.

Copy link

vercel bot commented May 13, 2024

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

A member of the Team first needs to authorize it.

Implement permanent links for user-generated content
@aprendendofelipe
Copy link
Collaborator

@mthmcalixto, obrigado pelo PR! 💪💪💪

Vou fechar porque essa funcionalidade não está nos planos de curto prazo.

A discussão em #1665 ainda está muito inicial e nem toca em pontos importantes como SEO, ou que razões os usuários teriam para trocar de nome, e se nesses casos seria melhor mesmo quebrar os links ou não.

@mthmcalixto
Copy link
Contributor Author

@aprendendofelipe entendo, quem sabe no futuro 💪💪💪

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.

Não quebrar links ao alterar nome de usuário
2 participants