sank20 / acra

Automatically exported from code.google.com/p/acra
0 stars 0 forks source link

persistent error notification #25

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Setup:
Acra 3.1.0 with error notification and email reporting

Steps to produce problem:
- cause an exception to get the error notification
- force the application to stop via settings -> applications -> manage 
applications -> force stop
- launch the app
- error notification comes again

I assumed the notification shouldn't come back at this point. Let's say the 
user doesn't want to report an error. But I don't see a way to clear the error 
other than using ErrorReporter.deletePendingReports(). So the error will always 
come back until the user gives up and reports it. Or am I missing something?

Original issue reported on code.google.com by martin.h...@gmail.com on 29 Dec 2010 at 4:18

GoogleCodeExporter commented 8 years ago
The current implementation in NOTIFICATION mode is that pending error reports 
are deleted when the user clicks the cancel button on the dialog. Ignoring the 
notification is not considered as a decision taken by the user to not report 
the crash.

Maybe we could change this in order to limit the bad feeling that can induce 
repeating error notifications.

Original comment by kevin.gaudin on 29 Dec 2010 at 4:41

GoogleCodeExporter commented 8 years ago
I would really like to see this change. Currently, I can delete pending reports 
on application start, but I don't want to delete reports that weren't sent 
simply because no internet connection was available.

Original comment by martin.h...@gmail.com on 29 Dec 2010 at 5:36

GoogleCodeExporter commented 8 years ago
Maybe you could offer an additional deletePendingNonSilentReports variation, 
one that deletes only those non-silent reports that haven't been acknowledged 
by the user yet (so that we keep reports that were previously acknowledged by 
the user but can't be sent yet because of a lacking internet connection).

Original comment by martin.h...@gmail.com on 30 Dec 2010 at 9:02

GoogleCodeExporter commented 8 years ago
I changed the deletePendingNonSilentReports method to 
deletePendingNonApprovedReports. Now it deletes reports which are neither 
silent nor approved. I don't think there is a need for a method which deletes 
reports which are non silent and both approved or not. In my opinion, approved 
reports should be treated the same way as silent reports. Comments welcome ;-)

Original comment by kevin.gaudin on 2 Jan 2011 at 3:13

GoogleCodeExporter commented 8 years ago
Perfect. Thanks for the quick response.

Original comment by martin.h...@gmail.com on 3 Jan 2011 at 5:50

GoogleCodeExporter commented 8 years ago
Issue 45 has been merged into this issue.

Original comment by kevin.gaudin on 4 Apr 2011 at 9:28

GoogleCodeExporter commented 8 years ago

Original comment by kevin.gaudin on 4 May 2011 at 11:55

GoogleCodeExporter commented 8 years ago

Original comment by kevin.gaudin on 6 Nov 2011 at 4:35