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

Adiciona lógica de decisão de denúncias de conteúdo. #187

Merged
merged 6 commits into from
Feb 13, 2024

Conversation

Luckvc
Copy link
Contributor

@Luckvc Luckvc commented Feb 12, 2024

Alcançado neste PR

Resolve #170

Criamos a lógica de decisão de remover ou não um conteúdo denunciado. Inserimos dois botões nos detalhes da denúncia.

Ao clicar Rejeitar denúncia, o status do report é mudado para rejected.

Ao clicar em Remover conteúdo, o conteúdo reportado recebe o status de removed.

  • No caso de uma publicação, o acesso a mesma fica bloqueado para todos.
  • No caso de um comentário, uma mensagem é exibida no lugar onde estaria este comentário

Adicionamos uma linha à validação do atributo published_at de post onde só valida se for um post agendado, já que do contrário essa data é inserida de forma interna.

Prints

  • Tela de detalhes da denúncia
    image

  • Denúncia decidida e conteúdo removido
    image

  • Comentário excluído
    image

Débitos

  • Ficou para o próximo PR a lógica de banimento de usuário.
  • Nem o admin tem acesso à um post removido no momento, exceto pela pré-visualização dentro da página de denúncia, pode ser algo que o time decida mudar no futuro.

rozbr96 and others added 4 commits February 9, 2024 17:59
Admin agora tem opções para remover conteúdo ou rejeitar denúncia
Posts removidos não podem mais ser acessados
Comentários removidos são substituídos por uma mensagem

Co-authored-by: Lucas Vasques <[email protected]>
Copy link
Member

@DanSmaR DanSmaR left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ficou muito legal! Value 💯

Copy link
Contributor

@gmkoeb gmkoeb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Boa pessoal, ficou bem completa essa PR, perfeito. Deixei alguns comentários, mas nada muito crítico 👌

spec/system/reports/user_report_spec.rb Show resolved Hide resolved
spec/models/post_spec.rb Outdated Show resolved Hide resolved
config/routes.rb Outdated Show resolved Hide resolved
@Luckvc Luckvc merged commit 19a931c into main Feb 13, 2024
2 checks passed
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.

[Reportar] Administrador toma decisão de análise de conteúdo na tela de feed de análise
4 participants