search

JeremyPP / Android.me-APP

Android app development for ndroidme.com
0 stars 1 forks source link

fix eventual notification bug when a new article is posted #7

Open MohHeader opened 10 years ago

MohHeader commented 10 years ago

@JeremyPP , Clarify please.

JeremyPP commented 10 years ago

@MohHeader When a new article is posted on the site, a notification should appears on the device telling that a new article is available. This has already been implemented, but it doesn't seems to work properly. Most of the times, there is no notification when a new article is posted.

JeremyPP commented 10 years ago

Also, it seems that sometimes when I post a new article on the site and I try to open the app, I get a Force Close and the app doesn't open. But I'm not really sure if this have something to do with the notification problem...

MohHeader commented 10 years ago

@JeremyPP , Do you have a ready way to test this issue ? i.e. Staging Server for Example or I have to simulate the webservice on my machine ?

JeremyPP commented 10 years ago

@MohHeader Unfortunately, the only way that I have to test this is when a new article is posted. If you have another way to simulate this locally, it would be much easier.

MohHeader commented 10 years ago

Ok, I will be simulating the issue locally :)

http://bbowden.tumblr.com/post/58650831283/accessing-a-localhost-server-from-the-genymotion

MohHeader commented 10 years ago

@JeremyPP , I did minor changes, So please let your eye on the issue :), and update me.

JeremyPP commented 10 years ago

I've posted a new article today, and I haven't seen any notification. To be honest, I think that the better thing to do would be to implement the notifications from the beginning again. Because I'm not sure that it was well implemented on the first time.

MohHeader commented 10 years ago

@JeremyPP , BTW What about adopting Push Notification instead of Pulling data from the server ? http://developer.android.com/google/gcm/index.html

JeremyPP commented 10 years ago

@MohHeader It seems to be a really better idea!

mikez23 commented 10 years ago

I think I have fixed this bug. I found an error with the notification service and fixed it.