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.

Sign up for GitHub

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

Jump to bottom

feat(edit profile): providing feedback to the user after editing their profile successfully #1545

Merged
merged 1 commit into from
Nov 8, 2023
Merged

feat(edit profile): providing feedback to the user after editing their profile successfully #1545

merged 1 commit into from
Nov 8, 2023

Conversation

viniciusamc
Copy link
Contributor

useRef was used for timeoutID and a variable was implemented to store the message value. The notification is triggered if the profile is saved successfully

#1524

@viniciusamc
feat(edit profile): providing feedback to the user after editing thei…
735084d 
…r profile successfully

useRef was used for timeoutID and a variable was implemented to store the message value. The
notification is triggered if the profile is saved successfully

filipedeschamps#1524
@vercel Vercel
Copy link

vercel bot commented Nov 2, 2023

@viniciusamc is attempting to deploy a commit to the TabNews Team on Vercel.

A member of the Team first needs to authorize it.

@viniciusamc
Copy link
Contributor Author

@aprendendofelipe
Decidi criar um novo pull request, enfrentei alguns problemas com o reset. Peço desculpas por ter complicado tanto um projeto que, na verdade, é bastante simples. Esta foi a minha primeira contribuição para um projeto open source e também a primeira vez que utilizei essas features

@vercel Vercel
Copy link

vercel bot commented Nov 6, 2023 •
edited

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

Name Status Preview Comments Updated (UTC)
tabnews ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 6, 2023 11:13am

@aprendendofelipe aprendendofelipe mentioned this pull request Nov 7, 2023
Merged
@aprendendofelipe
Copy link
Collaborator

@viniciusamc, pro favor, veja se está de acordo com o que eu fiz no PR #1548.

Seu commit foi aproveitado lá, mas fiz algumas modificações conforme coloquei na descrição.

Muito obrigado pela ajuda com a implementação 💪

@aprendendofelipe aprendendofelipe merged commit 735084d into filipedeschamps:main Nov 8, 2023
5 checks passed
@aprendendofelipe
Copy link
Collaborator

@viniciusamc, seu código já está rodando em produção! 🚀🚀🚀

Bem vindo à Turma de Contribuidores do TabNews! 👏👏👏

@aprendendofelipe
Copy link
Collaborator

@viniciusamc, você foi citado no post comemorativo:

https://www.tabnews.com.br/FelipeBarso/parabens-tabnews-aniversario-de-lancamento-e-mais-melhorias 🎉

@kaique-soares kaique-soares mentioned this pull request Nov 26, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@viniciusamc @aprendendofelipe