tabnews.com.br icon indicating copy to clipboard operation
tabnews.com.br copied to clipboard

Coins infinitas Bug

Open fgandraf opened this issue 2 years ago • 11 comments

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.

fgandraf avatar Nov 22 '22 13:11 fgandraf

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?

filipedeschamps avatar Nov 22 '22 13:11 filipedeschamps

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.

Rafatcb avatar Nov 22 '22 13:11 Rafatcb

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

brunofamiliar avatar Nov 22 '22 13:11 brunofamiliar

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

fgandraf avatar Nov 22 '22 14:11 fgandraf

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

brunofamiliar avatar Nov 22 '22 14:11 brunofamiliar

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.

filipedeschamps avatar Nov 22 '22 14:11 filipedeschamps

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 avatar Nov 22 '22 14:11 fgandraf

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 avatar Nov 22 '22 14:11 fgandraf

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)

fgandraf avatar Nov 22 '22 14:11 fgandraf

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) 🤝

filipedeschamps avatar Nov 22 '22 14:11 filipedeschamps

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.

votanruchel avatar Nov 23 '22 13:11 votanruchel

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

aprendendofelipe avatar Dec 14 '23 19:12 aprendendofelipe

@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".

ErickBessa avatar Jan 30 '24 20:01 ErickBessa

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.

aprendendofelipe avatar Jan 30 '24 22:01 aprendendofelipe