search

filipedeschamps / tabnews.com.br

Conteúdos para quem trabalha com Programação e Tecnologia.
https://tabnews.com.br
GNU General Public License v3.0
5.32k stars 388 forks source link

[UX] Melhorar mensagem de confirmação ao alterar `username` #895

Closed fgandraf closed 9 months ago

fgandraf commented 1 year ago

1-Fiz um comentário em uma postagem; 2-Resolvi mudar meu nome de usuário; 3-O comentário foi apagado mas as coins geradas foram mantidas.

Nao sei se esse era mesmo o comportamento esperado mas

"escrever comentário->mudar nome de usuário->escrever comentário->mudar nome de usuário....."

Resulta em coins sem participação efetiva.

filipedeschamps commented 1 year ago

Fala meu caro, muito interessante! Quando você quer dizer "comentário foi apagado", você usou o comando de apagar o comentário, ou a URL para o seu comentário mudou (e a antiga está dando 404)?

Qual seu usuário no TabNews para eu poder avaliar?

Rafatcb commented 1 year ago

Tentei reproduzir no ambiente de staging e ao apagar o comentário, as TabCoins eram removidas. Caso alguém queira testar, sugiro usar esse link (https://tabnews-1y5i5q50q-tabnews.vercel.app/) ao invés do site em produção.

brunofamiliar commented 1 year ago

Tentei reproduzir no ambiente de staging e ao apagar o comentário, as TabCoins eram removidas. Caso alguém queira testar, sugiro usar esse link (https://tabnews-1y5i5q50q-tabnews.vercel.app/) ao invés do site em produção.

Realmente em staging, quando removo o comentário o tabcoins são retirados. Eu não consegui acompanhar toda thread dessa feature #352; mas acredito que este é o comportamento correto @filipedeschamps certo? Porque senão pode gerar um "faturamento" falso de tabcoins, visto que tabcoins são ganhas apenas quando é gerado valor na plataforma

fgandraf commented 1 year ago

Usuário antigo: fgandraf Usuário novo: felipeGandra Postagem: https://www.tabnews.com.br/JhowBR/resumao-clean-code

@filipedeschamps Quando fiz o comentário e mudei o nome de usuário, ele apenas desapareceu da postagem. Então fiz novamente (gerando novas 2 coins). Agora, entrei para verificar e está aprecendo os dois.

image

brunofamiliar commented 1 year ago

Esse problema realmente vai acontecer, ainda não foi implementado uma tratativa; por isso o alerta:

image

O problema não é nas coins, e sim no comentário

filipedeschamps commented 1 year ago

Show turma, então é isso mesmo! Ao mudar o nome de usuário a URL absoluta antiga (com o username antigo) de fato deixa de existir, mas o conteúdo nesse caso não é apagado.

fgandraf commented 1 year ago

Esse problema realmente vai acontecer, ainda não foi implementado uma tratativa; por isso o alerta:

image

Eu vi esse alerta mas como fiz um "comentário" em uma publicação, não pensei que quebraria. De qualquer maneira, vou excluir um dos duplicados e verificar se as coins são recalculadas.

fgandraf commented 1 year ago

Pronto! exclui a duplicada e as coins foram recalculadas corretamente. Ficando então apenas a sugestão para mudar o texto para algo como "Isto irá quebrar todas as URLs das sua publicações e comentários" (ou "postagens e comentários") Valew turma!

fgandraf commented 1 year ago

Ah! talvez limitar a quantidade de vezes que um usuário pode mudar o nome (máximo 2 por exemplo, ou talvez, limitar por tempo: 1 alteração a cada X meses)

filipedeschamps commented 1 year ago

Isso é um excelente ponto de UX @fgandraf e quando escrevi essa mensagem, assumi de forma errada que as pessoas sabiam que cada publicação teu seu link único (basta clicar no horário da publicação) 🤝

votanruchel commented 1 year ago

Para os links quebrados acredito que invés de limitar o usuário a trocar o nickname x vezes, criar uma espécie de permalink na qual mesmo com o nickname antigo continuaria funcionando e os novos passariam a conter o novo nickname, claro que teria que implementar uma tratativa para um novo usuário não poder utilizar o nickname legado de outro usuário.

aprendendofelipe commented 11 months ago

Editei o título para corresponder ao que está pendente na issue, que é melhorar a mensagem aqui:

https://github.com/filipedeschamps/tabnews.com.br/blob/4e4d030cd4c0c2e240c96e5cfea16e43604aa223/pages/perfil/index.public.js#L81-L86

ErickBessa commented 9 months ago

@aprendendofelipe Gostaria de contribuir com essa issue, sabe se tem alguém atuando nela? Caso não tenha ninguém e eu siga com a modificação, o que acha da mensagem "Isto irá quebrar todas as URLs de seus conteúdos e comentários." ser implementada? Ela segue a sua sugestão de trocar "publicações" por "conteúdo" e segue a sugestão do @fgandraf de adicionar também "comentários".

aprendendofelipe commented 9 months ago

o que acha da mensagem "Isto irá quebrar todas as URLs de seus conteúdos e comentários." ser implementada?

"Conteúdos" já abrange publicações e comentários, mas pela motivação da issue, é melhor colocar "publicações e comentários". E acho que "Isso" fica mais correto no lugar de "Isto".

Isso irá quebrar todas as URLs de suas publicações e comentários.