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

Feat/marcar notificacoes como lida #201

Merged
merged 6 commits into from
Feb 13, 2024
Merged

Conversation

anaresgalla
Copy link
Contributor

@anaresgalla anaresgalla commented Feb 13, 2024

Esse PR aborda e resolve #186

Critérios de aceite:

  • Adapta e implementa testes para notificações
  • Mostra número de novas notificações não visualizadas no dropdown
  • Após visualização, o número de notificações no dropdown é zerado
  • Mostra notificações clicadas e não clicadas na página de notificações

image
Notificações clicadas e não clicadas

image
Número de notificações no dropdown

Débitos:

  • não há

hreis1 and others added 5 commits February 12, 2024 16:32
-Conta somente com status de não lida
-Ao clicar em notificações marca todas como lida

Co-authored-by: ana resgalla <[email protected]>
-remove coluna read
-adiciona coluna status

Co-authored-by: ana resgalla <[email protected]>
-Muda status da notificação para "clicked" ao ser clicada
-Muda status da notificação para "seen" ao acessar página de notificações

Co-authored-by: ana resgalla <[email protected]>
-corrige mensagem de tradução do mail no arquivo notification.pt-BR.ylm

Co-authored-by: ana resgalla <[email protected]>
Copy link
Contributor

@paulohenrique-gh paulohenrique-gh left a comment

Choose a reason for hiding this comment

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

Pra mim tudo certo, pessoal. Acho que ficaria bom também um botão pra limpar todas as notificações em um futuro PR 🚀

Copy link
Member

@DanSmaR DanSmaR left a comment

Choose a reason for hiding this comment

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

Boa, ficou dahora! 🥇

Recomendo aplicar o conceito de OOP como no do Likes Controller :)

app/controllers/notifications_controller.rb Show resolved Hide resolved
@hreis1 hreis1 merged commit 2da01f6 into main Feb 13, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Melhorias na página de notificações
5 participants