persistent error notification

[palpandi]

persistent error notification

[GoogleCodeExporter]

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]

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 [email protected] on 29 Dec 2010 at 5:36

[GoogleCodeExporter]

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 [email protected] on 30 Dec 2010 at 9:02

[GoogleCodeExporter]

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

• Changed state: Started
• Added labels: Milestone-Release3.2.0

[GoogleCodeExporter]

Perfect. Thanks for the quick response.

Original comment by [email protected] on 3 Jan 2011 at 5:50

[GoogleCodeExporter]

Issue 45 has been merged into this issue.

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

[GoogleCodeExporter]

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

• Added labels: Milestone-Release4.0
• Removed labels: Milestone-Release3.2.0

[GoogleCodeExporter]

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

• Changed state: Verified