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

[REVIEW] Página dando refresh ao enviar um comentário #610

Merged
merged 4 commits into from
Nov 6, 2018

Conversation

rodrigocam
Copy link
Contributor

@rodrigocam rodrigocam commented Nov 1, 2018

Descrição

  • Modificado função de exibição do toast de erro de comentário igual
  • Adicionado event listener para disparar exibição do toast quando o formulário é enviado e volta com erros

Issues Relacionadas

resolves: #589
resolves: #611

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

@codecov
Copy link

codecov bot commented Nov 1, 2018

Codecov Report

Merging #610 into develop will increase coverage by 0.1%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           develop     #610     +/-   ##
==========================================
+ Coverage    81.75%   81.85%   +0.1%     
==========================================
  Files          123      121      -2     
  Lines         3266     3207     -59     
  Branches       356      344     -12     
==========================================
- Hits          2670     2625     -45     
+ Misses         491      482      -9     
+ Partials       105      100      -5
Impacted Files Coverage Δ
src/ej_conversations/models/conversation.py 90.9% <0%> (-1.28%) ⬇️
src/ej_clusters/routes.py 96.11% <0%> (-0.04%) ⬇️
src/ej_reports/apps.py 100% <0%> (ø) ⬆️
src/ej_profiles/apps.py 100% <0%> (ø) ⬆️
src/ej/settings/__init__.py 100% <0%> (ø) ⬆️
src/ej_reports/fixes.py
src/ej_reports/roles.py
src/ej_clusters/models.py 79.71% <0%> (+0.14%) ⬆️
src/ej_boards/routes.py 90.65% <0%> (+1.36%) ⬆️
src/ej_reports/routes.py 64.8% <0%> (+20.35%) ⬆️
... and 1 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 1d01a74...bf01258. Read the comment docs.

Copy link
Collaborator

@fabiolamfleury fabiolamfleury left a comment

Choose a reason for hiding this comment

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

O toast aparece flutuando quando não tem a barra inferior e não desaparece como deveria

image

@fabiolamfleury fabiolamfleury changed the title [REVIEW] Página dando refresh ao enviar um comentário [WIP] Página dando refresh ao enviar um comentário Nov 5, 2018
@rodrigocam
Copy link
Contributor Author

rodrigocam commented Nov 6, 2018

@fabiolamfleury alterei o jeito que o toast aparece, agora ele vem de baixo e depois começa a descer até apagar, achoq que ficou legal. Também já merjei as suas alterações que estão na devel e corrigi dois bugs que encontrei!

@rodrigocam rodrigocam changed the title [WIP] Página dando refresh ao enviar um comentário [REVIEW] Página dando refresh ao enviar um comentário Nov 6, 2018
@rodrigocam rodrigocam force-pushed the i589_send_comment_bug branch from a0eca97 to bf01258 Compare November 6, 2018 13:18
@pablodiegoss pablodiegoss dismissed fabiolamfleury’s stale review November 6, 2018 13:18

Já Foi corrigido utilizando um animation pra subir e descer o toast

@pablodiegoss pablodiegoss merged commit 4fe153c into develop Nov 6, 2018
@delete-merged-branch delete-merged-branch bot deleted the i589_send_comment_bug branch November 6, 2018 14:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants