-
Notifications
You must be signed in to change notification settings - Fork 17
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
Browser based notifications #38
Comments
https://github.com/georgeosddev/react-web-notification seems to be a great place to start with, especially since it's react again. kudos++, if you could do this without adding a new dependency. |
@nkprince007 can you tell me which file deals with file upload? |
The backend is handled here: https://github.com/NITDgpOS/PiZilla/blob/master/server/routes.js#L16-L42 Once the upload has finished, this is where you can ask the client to push the notification. |
Actually, it's better and easier to use the HTML5 Notifications API directly. Refer MDN Docs on how to use it here. |
@nkprince007 please review my PR |
Added browser based notifications on successful file upload Fixes issue #38
Show browser based notifications on completion of file upload.
The text was updated successfully, but these errors were encountered: