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

Opção de habilitar/desabilitar notificações por email #720

Merged
merged 1 commit into from
Sep 9, 2022

Conversation

filipedeschamps
Copy link
Owner

Este PR adiciona a propriedade notifications ao objeto user e que pode conter o valor true (padrão) ou false. Esta propriedade é usada para o usuário decidir receber ou não a notificação por email de respostas em suas publicações. Junto deste PR também foi atualizada a página /perfil para o usuário conseguir controlar isso pelo client web.

E por hora, notifications é uma propriedade privada, ou seja, não será retornada no endpoint público do usuário que fica em /api/v1/users/[username] e só pode ser acessada pelo endpoint /api/v1/user que requer autenticação.

Em paralelo, eu fiquei aqui pensando em como já aproveitar essa migration para considerar planos futuros, por exemplo, outros tipos de notificação como SMS ou Push, mas vi que virou uma otimização prematura e melhor deixar para quando o sistema final de notificação for construído, onde eu imagino que o controle disso não ficará no objeto user e sim num outro objeto que conseguirá controlar a notificação de uma forma mais fragmentada, incluindo assinar por notificações de conteúdos de outras pessoas ou até qualquer tipo de publicação de qualquer outra pessoa.


Esta opção já pode ser vista na página /perfil em Homologação, mas não poderá ser utilizada enquanto não rodarmos a migration que está nesse PR.

@vercel
Copy link

vercel bot commented Sep 8, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
tabnews ✅ Ready (Inspect) Visit Preview Sep 8, 2022 at 8:12PM (UTC)

@filipedeschamps
Copy link
Owner Author

Migrations rodadas em Homologação e já pegou o default true 🤝

image

@filipedeschamps filipedeschamps merged commit 21e3ae4 into main Sep 9, 2022
@filipedeschamps filipedeschamps deleted the user-notifications branch September 9, 2022 02:06
@filipedeschamps
Copy link
Owner Author

Merged!! Let's goooo!!!

E já rodei a migration em Produção 🤝

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.

2 participants