gitskarios / Gitskarios

GitHub Android application
MIT License
527 stars 73 forks source link

double notifications and it seem much noisier on latest release #646

Open twig opened 7 years ago

twig commented 7 years ago

I've been using it for a while and it seems nosier than the past.

I've resorted to disabling notifications altogether because of the sound. Please make it optional.

I also seem to be getting double notifications.

1 when I get told about new alerts and a second one when I tap on that notification and then another notification when the app fetches alerts, playing the same "new alert" sound agaon

alorma commented 7 years ago

Hi

I've disabled the sound for notifications.

alorma commented 7 years ago

I not reproduce the double notifications, what android version are you using?

webmaster33 commented 7 years ago

I don't get double notifications under Android 6.0.1, nor 4.1.1. Working fine for me.

However I get too few notifications, and I always get email much earlier, than to get notification from the app.

Note, when I install a Gitskarios update, I don't get notification at all, until I start Gitskarios once. But this true for other apps, too, like Facebook.

alorma commented 7 years ago

Yes, notifications cannot be sent via push.

App does two things:

1) Check for notifications every 30 minutes. 2) Get push of a notifications, if you add a webhook to your repositories:

--> https://cryptic-ravine-97684.herokuapp.com/message

webmaster33 commented 7 years ago

Personally I like notification sound.

alorma commented 7 years ago

@webmaster33 I will add an option to enable / disable it

twig commented 7 years ago

Thanks for your work on it @alorma

I'll try to find a way to record the double notification bug.