-
Notifications
You must be signed in to change notification settings - Fork 396
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
Criar endpoints e páginas para listar eventos de firewall e ações da moderação #1660
Comments
Estava pensando em como poderíamos adicionar links na UI para as novas páginas. Com a implementação desse issue, provavelmente surgirão duas páginas que podem ter links no menu (por exemplo, Uma opção é colocar os links no menu principal (ao lado de Novo.menu.1.webmAcho que não fica bom adicionar tudo no O que vocês acham mais adequado? Talvez, por essas funcionalidades serem "menos acessadas", deixar no menu hambúrguer? Ou então, deixar no menu hambúrguer apenas o Essa opção me parece melhor do que a primeira. O link do Firewall será exibido apenas para quem tem permissão. Ele poderia ficar no rodapé, mas acredito que essa ação não precise ficar tão escondida, pois pode ser útil para o moderador acessar de vez em quando e acompanhar o que está acontecendo. Uma possibilidade também é deixar a página "Ações da moderação" acessível pela página de Status, já que são assuntos mais ou menos relacionados, mas precisaria pensar onde colocar o link. Talvez embaixo da Uma outra opção é não ter página para |
Também acho que é uma boa ideia não ir além de uma pequena lista na página de status na primeira versão da lista de eventos. Pode não ser uma quantidade fixa de eventos, mas um intervalo de tempo determinado, talvez dois dias ou um pouco mais. Talvez seja ainda melhor mostrar uma tabela com a contagem por tipos de eventos de moderação no período. Essa lista é mais para um usuário qualquer poder investigar o que pode ter acontecido com alguma publicação ou conta que deixou de existir. Por sigilo de dados, não dá para dar muitos detalhes nessa lista pública, então acho que a notificação para usuários moderados é algo mais importante. No #1638 já lidamos com os eventos de firewall, mas é preciso notificar os usuários afetados nos outros tipos de moderação. |
Descrição
O PR #1638 criou um endpoint
GET /api/v1/events/firewall/[id]
para obter dados relacionados à um evento gerado pelo firewall interno. Em #1638 (comment), surgiu a ideia de criar novos endpoints relacionados à ações da moderação:GET /api/v1/events/firewall
: similar ao mencionado anteriormente, porém retorna todos os eventos ao invés de um único. Deve verificar a featureread:firewall:list
e ter paginação.GET /api/v1/events/moderations
: dados que podem ser públicos sobre as atuações da moderação. Isso precisa ser melhor debatido para entender o que faz sentido exibir e como exibir.GET /api/v1/events/moderations/[id]
: dados restritos aos moderadores sobre cada uma das atuações da moderação.Além dos novos endpoints, precisaremos de novas páginas para exibir esses dados.
Sugestão de implementação
Podemos criar uma página para a lista de eventos de firewall e outra para a lista de eventos da moderação. Talvez seja necessário criar novas páginas para a visualização de um evento específico do firewall e para uma ação específica da moderação, mas podemos experimentar uma visualização simplificada (hoje não temos nenhum tipo de funcionalidade que a visualização das informações não seja numa página específica).
A página com detalhes do evento do firewall pode ter um botão para "desfazer", caso o usuário tenha permissão para isso, já que após o merge do PR #1638 essa ação será feita diretamente pela API, sem uma UI.
As funcionalidades aqui podem ser divididas em mais de um PR, principalmente porque o endpoint
1
é mais simples, dado que oGET /api/v1/events/firewall/[id]
já está pronto (ainda falta o merge do PR), e que os endpoints2
e3
precisam de discussões sobre o que será retornado. Além disso, o backend e o frontend podem ser feitos em PRs separados também, por pessoas diferentes e em momentos diferentes.The text was updated successfully, but these errors were encountered: