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

MODEL:CONTENT:CREDIT_OR_DEBIT_TABCOINS:NEGATIVE_USER_EARNINGS #1540

Open
samsepiol1 opened this issue Oct 22, 2023 · 2 comments
Open

MODEL:CONTENT:CREDIT_OR_DEBIT_TABCOINS:NEGATIVE_USER_EARNINGS #1540

samsepiol1 opened this issue Oct 22, 2023 · 2 comments
Labels
back Envolve modificações no backend

Comments

@samsepiol1
Copy link

inspecior

Olá a todos, boa tarde. Criei uma nova conta no TabNews, visto que a minha anterior foi excluída, e comecei a repostar artigos antigos que eram da outra conta. Em uma das publicações, recebi uma avaliação negativa, o que acabou me impedindo de comentar ou publicar novos conteúdos. Pelo que verifiquei esse código que aparece é referente ao error_id.

Interessantemente, consegui publicar tanto o comentário quanto o post alguns longos minutos depois

@samsepiol1
Copy link
Author

samsepiol1 commented Oct 22, 2023

A regra de negócio que faz essa verificação se encontra na pasta dos models em content.js

      throw new ForbiddenError({
        message: 'Não é possível publicar porque há outras publicações mal avaliadas que ainda não foram excluídas.',
        action: 'Exclua seus conteúdos mais recentes que estiverem classificados como não relevantes.',
        errorLocationCode: 'MODEL:CONTENT:CREDIT_OR_DEBIT_TABCOINS:NEGATIVE_USER_EARNINGS',
      });
    }

Mesmo que seja uma regra de negócio válida, essa parte apenas faz a verificação do saldo do usuário e retorna o erro_id, acredito que o ideal seria retornar qual publicação foi negativa para que possa ser feito a sua retirada

@Rafatcb Rafatcb added the back Envolve modificações no backend label Dec 16, 2023
@aprendendofelipe
Copy link
Collaborator

No seu caso havia apenas um conteúdo negativado e, se entendi direito, logo foi positivado, então não foi necessário apagar. É isso?

Se a mensagem estivesse dizendo para apagar "certa publicação", poderia ter feito você excluí-la sem necessidade.

Imagina se alguém tem mais de um conteúdo negativado, e outros positivados, de forma que com a exclusão de qualquer um deles o saldo já seria suficiente para poder criar novas publicações. Se a gente listar os negativados, pode induzir a exclusão de todos, mas quem deveria decidir isso é o próprio autor.

A mensagem é apenas para informar um autor de que ele deve revisar os seus conteúdos recentes. E dar mais uma oportunidade da pessoa procurar saber o que é esperado das publicações no TabNews, já que muitos ignoram o aviso que aparece antes da primeira publicação.

Não acho que devemos listar as publicações negativadas.

Já a mensagem do erro sempre pode ser melhorada. Inclusive direcionar para o FAQ quando ele for implementado 👍

@Rafatcb Rafatcb mentioned this issue Dec 23, 2023
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
back Envolve modificações no backend
Projects
None yet
Development

No branches or pull requests

3 participants