ejplatform / ej-server

EJ uses gamification and machine learning to promote discussions and participation in a virtual democracy

https://ejplatform.github.io/ej-server/

Other

38 stars 17 forks source link

[REVIEW] Issue 526 Configurações de notificações inbox (front) #595

Closed lucashm closed 5 years ago

lucashm commented 5 years ago

Descrição

Implementa a página de configuração de notificações inbox.

Issues Relacionadas

Resolves #526

Checklist

[ ] Os commits seguem o padrão do projeto (Flake8 e afins)
[ ] Os testes estão passando e cobrem as mudanças

Imagens/Comentários

Atualmente, a página está da seguinte forma:

Referencial do invision: https://projects.invisionapp.com/d/main#/console/14872865/324346885/preview

Obs: PR travado, no aguardo do PR #417

codecov[bot] commented 5 years ago

Codecov Report

Merging #595 into develop will decrease coverage by 0.24%. The diff coverage is 50%.

@@             Coverage Diff             @@
##           develop     #595      +/-   ##
===========================================
- Coverage    78.52%   78.28%   -0.25%     
===========================================
  Files          141      142       +1     
  Lines         3996     4030      +34     
  Branches       448      452       +4     
===========================================
+ Hits          3138     3155      +17     
- Misses         724      741      +17     
  Partials       134      134

Impacted Files	Coverage Δ
src/ej/settings/apps.py	`65.21% <ø> (ø)`	:arrow_up:
src/ej_users/models.py	`97.82% <100%> (+0.15%)`	:arrow_up:
src/ej_notifications/forms.py	`100% <100%> (ø)`
src/ej_notifications/routes.py	`31.42% <11.76%> (-18.58%)`	:arrow_down:
src/ej_users/rules.py	`92.3% <75%> (-7.7%)`	:arrow_down:

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 ce0bae6...b02de57. Read the comment docs.

lucashm commented 5 years ago

Issue implementada parcialmente. Algumas pendências:

Definir notificação como vista ou não. (@fabiolamfleury aquele campo de status da mensagem é pra isso? Seria legal se ele fosse definido como visto ou pendente apenas, não uma string solta!)
Como não dá pra saber se uma notificação foi vista ou não, também não foi implementado o número de notificações pendentes.
As configurações de notificações passadas pelas telas do invision não batiam com as opções definidas no servidor. No momento, as do servidor estão nessa versão, enquanto as do invision estão comentadas e estáticas.

lucashm commented 5 years ago

Tela:

Ah, outro detalhe, essa imagem ao lado da notificação é a mesma para todos, ela deveria conter ícones diferentes para cada tipo de notificação, certo? Onde que está documentado essa diferenciação? Uma lista com os respectivos iria ajudar muito :) @ricardopoppi

ricardopoppi commented 5 years ago

@lucashm respondendo sua pergunta, nessa versão só teremos imagem para as notificações de poder. creio que os outros tipos não foram implementados e não dará tempo. temos portanto dois tipos de notificações de poder, (i) ponte de diálogo e (ii) ativista de minoria.

ambas as imagens estão disponíveis no wireframe mencionado na issue: https://projects.invisionapp.com/d/main#/console/14872865/324346885/preview

qq coisa estou na escuta

lucashm commented 5 years ago

update:

lucashm commented 5 years ago

Ok, tudo certo creio... (assim que passar os testes saberemos!) No mais, ficou faltando apenas os ícones das notificações... Do jeito que está configurado no back, cada notificação teria um ícone voltado para o seu tipo de canal, como a Fabíola mencionou ali. É uma ideia possível de se implementar no momento, mas como cada canal não tem um ícone próprio definido, deixei essa tarefa como pendente. Segue os tipos de canais: É preciso uma discussão com relação à como esses ícones devem se encaixar nas notificações... Mas no geral, já vale o PRzão! Ativar modo review!