[REVIEW] Create basic models for EJ powers

[fabiolamfleury]

Descrição

Criação dos poderes do ponte de diálogo e ativista de minoria

Issues Relacionadas

resolves: #437

Checklist

• [x] Os commits seguem o padrão do projeto (Flake8 e afins)
• [x] Os testes estão passando e cobrem as mudanças
• [x] Marcou no título do pull request se ele é work in progress [WIP] ou se está pronto para revisão [REVIEW]

[codecov[bot]]

Codecov Report

Merging #485 into develop will increase coverage by 0.45%. The diff coverage is 84.61%.

@@             Coverage Diff             @@
##           develop     #485      +/-   ##
===========================================
+ Coverage    81.57%   82.02%   +0.45%     
===========================================
  Files          123      129       +6     
  Lines         3273     3400     +127     
  Branches       358      370      +12     
===========================================
+ Hits          2670     2789     +119     
- Misses         497      504       +7     
- Partials       106      107       +1

Impacted Files	Coverage Δ
src/ej/settings/apps.py	65.21% <ø> (ø)	:arrow_up:
src/ej_powers/__init__.py	100% <100%> (ø)
src/ej_powers/models.py	100% <100%> (ø)
src/ej_powers/admin.py	100% <100%> (ø)
src/ej_powers/rules.py	65.38% <55.55%> (ø)
src/ej_conversations/rules.py	76.11% <60%> (-1.3%)	:arrow_down:
src/ej_powers/functions.py	84.61% <83.78%> (ø)
src/ej_powers/apps.py	100% <0%> (ø)
... and 6 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 6760713...97eb418. Read the comment docs.

[fabiolamfleury]

Problema encontrado na dependência django-jsonfield, não é compatível com o django 2 (utilizando em sqlite) estou procurando algo que possa substituí-lo

(CORRIGIDO: Foi utilizado um picklefield no lugar, que é database agnostic)

[fabiolamfleury]

Implementações que foram mapeadas que faltam para que esse PR esteja completo:

• [x] Criar no arquivo functions.py funções para criações dos poderes GivenBridgePower e GivenMinorityPower (facilitando o uso, para não usar a model diretamente)

• [x] Criar um constraint do poder para este ficar relacionado não só a um usuário mas também a uma conversa

• [x] Atualizar implementações das rules

• [x] Alterar implementação do use_power das duas models (bridge e minority) para verificar se o comentário que está sendo promovido é de fato da conversa que o usuário possui poder

• [x] Criar função que retorna todos os comentários promovidos para aquele usuário em determinada conversa e alterar a regra que retorna um comentário para o usuário votar para usar essa função (deverá ativar a 'limpeza' de comentários promovidos já expirados)

• [ ] * existe alguma situação na qual a promoção do comentário não irá expirar?

Qual o tempo default para expirar um poder? E para expirar a promoção de um comentário?

[rodrigocam]

@fabiolamfleury como funcionaria para dar poder para um usuário? Por exemplo: O usuário jubileu foi selecionado para ter o poder de promover conversas pelo ML. No caso eu usaria a função give_promotion_power(user=jubileu)? ou ele só pode promover uma conversa específica? Por que a função pede mais 3 argumentos, sendo um deles power_class?

[fabiolamfleury]

A função give_promotion_power(user=jubileu) é usada apenas internamente no arquivo, para dar poder para um usuário deve ser give_minority_power() ou give_bridge_power(), passando como argumento o usuário que receberá o poder, a conversa que ele terá esse poder e quais usuários serão afetados por ele. Atualizei o comentário de descrição das funções

[rodrigocam]

@fabiolamfleury Para aceitar falta só criar uma regra para checar se um usuário pode promover comentários!

[fabiolamfleury]

@rodrigocam atualizei as regras mas todas estão marcadas apenas como predicado, haveria necessidade de mais definições?

[rodrigocam]

@fabiolamfleury acho que tá tranquilo por enquanto!