[REVIEW] Bug modais

[LeoSilvaGomes-zz]

Descrição

• [x] Modais só devem aparecer no primeiro acesso do usuário

• [x] Botão "NEXT" e "continue participating" não geraram strings de tradução

• [x] Botão "NEXT" e setinhas do slide (>) não estão funcionando na primeira carga. Só funcionam se recarregarmos a página

• [x] Modais não estão de acordo com wireframes (tamanho, alguns estilos, três pontinhos abaixo): https://projects.invisionapp.com/d/main#/console/14872865/324346881/preview https://projects.invisionapp.com/d/main#/console/14872865/324346880/preview https://projects.invisionapp.com/d/main#/console/14872865/324346879/preview

Issues Relacionadas

678

Checklist

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

Imagens/Comentários

Antes

Depois

[LeoSilvaGomes-zz]

"Modais só devem aparecer no primeiro acesso do usuário"

@ricardopoppi Gostaria de saber se podemos fazer as modais aparecerem no momento em que a pessoa fizer o login ao invés de ser quando a pessoa entra na conversa

[ricardopoppi]

@LeoSilvaGomes acho que essa abordagem dos modais aparecerem para o usuário logo após o login, e uma única vez é ótima. Pode seguir em frente assim.

[codecov[bot]]

Codecov Report

Merging #686 into develop will not change coverage. The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #686   +/-   ##
========================================
  Coverage    78.52%   78.52%           
========================================
  Files          141      141           
  Lines         3996     3996           
  Branches       448      448           
========================================
  Hits          3138     3138           
  Misses         724      724           
  Partials       134      134

---

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 16176bc...3303e6b. Read the comment docs.

[codecov[bot]]

Codecov Report

Merging #686 into develop will increase coverage by 0.24%. The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop     #686      +/-   ##
===========================================
+ Coverage    78.32%   78.57%   +0.24%     
===========================================
  Files          142      141       -1     
  Lines         4051     4004      -47     
  Branches       455      449       -6     
===========================================
- Hits          3173     3146      -27     
+ Misses         744      724      -20     
  Partials       134      134

Impacted Files	Coverage Δ
src/ej_profiles/routes.py	98.03% <ø> (ø)	:arrow_up:
src/ej_boards/routes.py	89.28% <ø> (ø)	:arrow_up:
src/ej_conversations/routes/conversations.py	96.49% <100%> (+0.41%)	:arrow_up:
src/ej_users/routes.py	70% <100%> (+0.5%)	:arrow_up:
src/ej_conversations/roles.py	80% <0%> (-1.67%)	:arrow_down:
src/ej_conversations/models/conversation.py	87.14% <0%> (-1.38%)	:arrow_down:
src/ej_users/models.py	97.67% <0%> (-0.16%)	:arrow_down:
src/ej_conversations/routes/admin.py	100% <0%> (ø)	:arrow_up:
src/ej/settings/apps.py	65.21% <0%> (ø)	:arrow_up:
src/ej_notifications/forms.py
... and 4 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 0e01738...254361b. Read the comment docs.