Can't delete crashes / or group them!

ACRA / acralyzer

Open source backend for ACRA reports.

GNU General Public License v3.0

489 stars 90 forks source link

Can't delete crashes / or group them! #34

Closed expresspotato closed 11 years ago

expresspotato commented 11 years ago

Yikes - this is almost a deal breaker for me. There should really be a way to delete crashes or group them somehow. How can I tell which are old and which are new?

Delete is preferred and would probably only take you a few minutes to implement!

Please :+1:

johnuopini commented 11 years ago

Grouping by exception would be awesome

KevinGaudin commented 11 years ago

Grouping is being developed and discussed in issue #2.

Adding a single report delete button is easy indeed. Might be doing it... or not. In my mind it's in some way linked to grouping because there are several kinds of deleting motivations:

delete reports because the issue is solved => this requires grouping to be efficient. You would mark group of reports as solved to hide them, and they would still appear in the dashboard graphs.
delete reports because you just want to recover some storage space computation time => a 'delete reports older than X days' feature would be more efficient. => after deleting they couldn't be visible in the dashboard graphs

Do you think that a per-report delete button is necessary if the 2 previous points are implemented ?

johnuopini commented 11 years ago

Ok so in this case also #35 is a dupe because is partially problem "2"

expresspotato commented 11 years ago

For me just a simple delete is all I'm asking... Seems really primitive not to have it.

Kevin

On Wed, May 1, 2013 at 5:21 PM, johnuopini notifications@github.com wrote:

Ok so in this case also #35 https://github.com/ACRA/acralyzer/issues/35is a dupe because is partially problem "2"

— Reply to this email directly or view it on GitHubhttps://github.com/ACRA/acralyzer/issues/34#issuecomment-17307313 .

KevinGaudin commented 11 years ago

I've been working on a "purge" branch, implementing the following:

delete one report
admin panel with:
- delete reports older than X days
- delete reports from app version code X and below

The branch is available here, you can test it before I merge it to the master branch. This could happen at the end of the week.

KevinGaudin commented 11 years ago

If you want to test the purge branch, you need to update your acra-storage database with the latest master commit.

johnuopini commented 11 years ago

If someone is interested i have slightly modified the PHP proxy code to allow filtering by version (so if you release a new APK you can filter out old releases saving POSTs requests on iriscouch and also cleaning up the bugs you get), code is here http://pastebin.com/PT8VqtnP

KevinGaudin commented 11 years ago

About deleting/purging, the master branch now includes the previously announced features:

delete one report
admin panel with:
- delete reports older than X days
- delete reports from app version code X and below