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

Add logout feature #60

Merged
merged 3 commits into from
Feb 2, 2022
Merged

Add logout feature #60

merged 3 commits into from
Feb 2, 2022

Conversation

fanny
Copy link
Contributor

@fanny fanny commented Dec 21, 2021

Que problema está resolvendo?

Notion Task

Como pode ser manualmente testado?

Logue-se no sistema, após isso, clique no ícone de Logout na parte inferior do sistema.

Screenshots

Tipos de mudanças

  • Conserta um bug
  • Nova funcionalidade
  • Refatoração de código
  • Atualiza documentação

@netlify
Copy link

netlify bot commented Dec 21, 2021

✔️ Deploy Preview for na-mesa ready!

🔨 Explore the source changes: 4324445

🔍 Inspect the deploy log: https://app.netlify.com/sites/na-mesa/deploys/61fb0afbfa69ea0008c22f11

😎 Browse the preview: https://deploy-preview-60--na-mesa.netlify.app

@vinifarias
Copy link
Member

@fanny eu não encontrei o botão de logout na tela 🤔

image

@fanny
Copy link
Contributor Author

fanny commented Dec 22, 2021

@fanny eu não encontrei o botão de logout na tela 🤔

image

@vinifarias é na tela do cliente

@fanny
Copy link
Contributor Author

fanny commented Dec 22, 2021

É a task #42 @vinifarias

@vinifarias
Copy link
Member

@fanny eu realmente não lembro de a gente ter discutido essa ação para o client, lembro apenas para o admin.
O que tu acha de deixar ela em draft? Porque penso que a gente precisa pensar melhor no que acontece quando, por exemplo, o usuário utiliza o logout tendo feito pedidos.

@thayannevls thayannevls merged commit d37efdc into main Feb 2, 2022
@thayannevls thayannevls deleted the feat/edit-screen branch February 2, 2022 22:56
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.

3 participants