bunqCommunity / bunqDesktop

The unofficial, free and open source desktop application for the bunq API
https://bunqdesk.top
MIT License
272 stars 53 forks source link

Notifications #19

Closed DennisSnijder closed 5 years ago

DennisSnijder commented 6 years ago

In the near future we want to support notifications for the BunqDesktop client.

This issue is to discuss on how we should approach this since the API only allows callbacks at the moment.

ntimo commented 6 years ago

Maybe you could create a helper application that calls the api for updates and if there is sone shows the user a notification. The helper could be started at system boot without having to start the BunqDesktop app itself.

DennisSnijder commented 6 years ago

@ntimo this could be one solution indeed. Maybe we could do something like setting up a simple server which catches the notifications and transmits the events to the correct client via sockets. (Would be really easy with https://pusher.com)

But again, that kinda kills the idea of why we switched to a desktop application.

ntimo commented 6 years ago

@DennisSnijder First I think the notification checking should be done local on the device because of the privacy. And also using a service like this would cost you money/the user. And that is not a perfekt solution in my eyes.

DennisSnijder commented 6 years ago

@Crecket suggestions on this topic? 😝

Crecket commented 6 years ago

The most reasonable and secure solution would be to pull every 10-30 seconds (Perhaps make this configurable/opt-in?)

ntimo commented 6 years ago

@Crecket yes, that would be great +1

Crecket commented 6 years ago

I want to add that we already have a component I used to display default Chrome notifications found in src/react/Helpers/Notification.js. This should be able to show notifications even when the app is in the background

basst85 commented 6 years ago

You can also create a system tray icon functionality to keep the app alive.

DennisSnijder commented 6 years ago

Small update on this topic. We requested an "/events" endpoint to the Bunq dev team. (Lets hope they will implement it 😄 )