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

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

Closed
fgandraf opened this issue Nov 22, 2022 · 14 comments · Fixed by #1617
Closed

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

fgandraf opened this issue Nov 22, 2022 · 14 comments · Fixed by #1617
Labels
front Envolve modificações no frontend good first issue Boa para quem quer começar a contribuir, mas pode ser implementado por qualquer senioridade

Comments

@fgandraf
Copy link

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
Copy link
Owner

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
Copy link
Collaborator

Rafatcb commented Nov 22, 2022

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
Copy link
Contributor

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
Copy link
Author

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
Copy link
Contributor

brunofamiliar commented Nov 22, 2022

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
Copy link
Owner

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
Copy link
Author

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
Copy link
Author

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
Copy link
Author

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
Copy link
Owner

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
Copy link

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 aprendendofelipe changed the title Coins infinitas Bug [UX] Melhorar mensagem de confirmação ao alterar username Dec 14, 2023
@aprendendofelipe
Copy link
Collaborator

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

const confirmChangeUsername = await confirm({
title: `Você realmente deseja alterar seu nome de usuário?`,
content: `Isto irá quebrar todas as URLs das suas publicações.`,
cancelButtonContent: 'Cancelar',
confirmButtonContent: 'Sim',
});

@aprendendofelipe aprendendofelipe added front Envolve modificações no frontend good first issue Boa para quem quer começar a contribuir, mas pode ser implementado por qualquer senioridade labels Dec 14, 2023
@ErickBessa
Copy link
Contributor

ErickBessa commented Jan 30, 2024

@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
Copy link
Collaborator

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.

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 good first issue Boa para quem quer começar a contribuir, mas pode ser implementado por qualquer senioridade
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants