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

Iago/notif nativas #80

Open
wants to merge 20 commits into
base: develop
Choose a base branch
from
Open

Iago/notif nativas #80

wants to merge 20 commits into from

Conversation

14g0
Copy link
Member

@14g0 14g0 commented Apr 6, 2024

Implementação de notificações nativas 🔔

Refatoração do Contexto do PopUp.

  • ✅ Ao invés de importar as funções de contexto do PopUp e adicionar também a importação do componente do PopUp, o componente assim como alguns dos componentes estilizados para que quando desejados fossem importados pelo contexto.
  • ⛔ Percebi que os parâmetros do componente do PopUp não são mais identificados pelo intelisense 😢.

Conserto do PopUp de instalação de IOS

  • ✅ Com as alterações mencionadas acima, foi necessário refatorar todos os pedaços do código que possuíam o PopUp e isso gerou um bug no PopUp de lançamento porque quando ele era fechado, ele abria novamente na mesma hora.
  • ✅ Removi alguma estilização no DownPop que não impactava em nada.
  • ✅ Removido o contexto do DownPop já que nunca houve necessidade.
  • ✅ O DownPop foi adicionado aos componentes globais, a restrição que ele sofre é não poder aparecer no /Tutorial que é o que está acontecendo. Eu acho interessante ele aparecer em /Restaurante.

Front pronto para notificações.

  • ✅ Utilização do pacote da Firebase para inscrição no serviço de Push da mesma. O token referente a instância do aplicativo é armazenado no localStorage para que a aplicação não faça a requisição de cadastro do token no tópico e a solicitação do token toda vez.
  • ✅ Criação de arquivos e modificação de funções referentes ao Service Worker.

@14g0 14g0 marked this pull request as ready for review April 6, 2024 03:57
@14g0 14g0 requested a review from carloseduardobanjar April 6, 2024 22:00
@14g0 14g0 force-pushed the iago/notifNativas branch from a0bcc7e to ca96905 Compare April 30, 2024 00:53
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