Arquisoft / radarin_en3a

2 stars 1 forks source link

Notifications in the webapp #83

Closed irenebello closed 3 years ago

irenebello commented 3 years ago

We have a problem passing the friend list to de restapi function. The funtion works properly in the postman.

UO269509 commented 3 years ago

I create a new branch to solve this problem (WebNotification), first of all I tried to send a hardcoded notification using the react-toastify library and it seems to work. The next step is to solve the parameters we are passing to the restapi.

UO269509 commented 3 years ago

Now, he have notifications, the problem was that we return a json instead of text.

irenebello commented 3 years ago

I fixed some minor errors we had with the restapi and the webapp. Now the notifications are properly working

irenebello commented 3 years ago

Right now the basic funcionality of the notifications is implemented. Now I am trying to do something similar to the github bell button that shows when we have a notification in our app.

irenebello commented 3 years ago

Now we have in the nav a bell that tells us if we have some notification and each 30 seconds we can see the if we have a near friend. Now the only problem is to have it visually good and to solve a compiling error

irenebello commented 3 years ago

@pglez82 I have a compiling error that I don´t know how to solve. The screen is the next one. How can I fix this? imagen

pglez82 commented 3 years ago

Hi. It is not an error but a warning. The same problem is described here: https://stackoverflow.com/questions/55840294/how-to-fix-missing-dependency-warning-when-using-useeffect-react-hook

If nothing works, you can always disable the warning as described in the stackoverflow post.

irenebello commented 3 years ago

Fixed the error with the warning suppressing it, and also did that only one notification appeared in the window (before two notifications appeared). Now we only need to put the visual correct. But the notifications are done.