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

Added Toast.jsx #210

Closed
wants to merge 1 commit into from
Closed

Added Toast.jsx #210

wants to merge 1 commit into from

Conversation

akshit-bhutani-19
Copy link
Contributor

Created Toast.jsx using react-hot-toast
It will give a toast for loading and change to sucess or failure.
Function in Toast.jsx can be used with a promise on API call. Message has to be specified in API call using resolve or reject.

Implementation has been specified in comments of Toast.jsx

Toast.mp4

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Great job, @Akki019! 🎉 Thank you for submitting your pull request. Your contribution is valuable and we appreciate your efforts to improve our project.

We will promptly review your changes and offer feedback. Keep up the excellent work! Kindly remember to check our contributing guidelines

Copy link

Hi there! This issue is still open. We are looking forward to your response.
Assignees: None

Copy link
Contributor

@thecuriousteam thecuriousteam left a comment

Choose a reason for hiding this comment

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

Please add some clean code by removing and keeping the right commits @Akki019

Copy link

Hi there! This issue is still open. We are looking forward to your response.
Assignees: @Akki019

2 similar comments
Copy link

Hi there! This issue is still open. We are looking forward to your response.
Assignees: @Akki019

Copy link

Hi there! This issue is still open. We are looking forward to your response.
Assignees: @Akki019

@akshit-bhutani-19 akshit-bhutani-19 closed this by deleting the head repository May 23, 2024
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