Curious-Ecosystem / Curious-Connect

This is a sub product of Curious Ecosystem created to have one on one and one on many live streaming sessions to support students and organizations
BSD 4-Clause "Original" or "Old" License
76 stars 127 forks source link

Global API Error Handler (Toast.jsx) #86

Open CDCNaman opened 4 months ago

CDCNaman commented 4 months ago

Feature Description

Create a toast for displaying error message from API

Problem or Opportunity

This component should be used whenever a error message needs to be displayed

Proposed Solution

You can also use libraries such as react-hot-toast

Additional Context

image Please let us know if you have any other library in your mind other than react-hot-toast https://react-hot-toast.com/

Instructions

Stay you can start working on it by taking latest pull from main branch

Respect Co Developers And Please Reach-out to admins if any issues

akshit-bhutani-19 commented 4 months ago

Can you assign this to me?

Tomkndn commented 4 months ago

I would like to contribute to this project, can you assign me this issue. As I know one more popular react notification library known as react-toastify (https://www.npmjs.com/package/react-toastify)

CDCNaman commented 4 months ago

@Tomkndn @Akki019 Both of you can start working on it.

Tomkndn commented 4 months ago

Should I use react-toastify or your mentioned library react-hot-toast?

CDCNaman commented 4 months ago

Should I use react-toastify or your mentioned library react-hot-toast?

@Tomkndn you can use react-toastify as well

Tomkndn commented 4 months ago

When I started my dev both frontend and backend server I am seeing the frontend part without the functionality. Without functionality how can i add toast to error handling on API.

dhyaneshsiddhartha15 commented 4 months ago
thecuriousteam commented 4 months ago

any updates @Tomkndn @Akki019 @dhyaneshsiddhartha15

akshit-bhutani-19 commented 4 months ago

I have created it and testing right now. But would need a review on this

thecuriousteam commented 4 months ago

For Sure @Akki019

dhyaneshsiddhartha15 commented 4 months ago

@thecuriousteam I'm currently working on it and testing will make it done soon.