-
Notifications
You must be signed in to change notification settings - Fork 623
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Sorting of events at /admin/reports #220
Comments
+1 for additional sorting (and even better filtering) |
To add another complication here: reports in dashboard 'Recent reports' is done by incident_dateadd This is confusing and needs to be more obvious |
Admin sorting is now merged into develop. |
@rjmackay close this out since its done and create a new one for extension into frontend? |
Adding reports search form to admin/reports pages. Searching by: * Category * Approved * Verified * Keyword * Location * Media type * Mode (SMS/Web/Twitter) * Date
…ahidi#220 This makes sure we still get results if from and to dates are the same. Also more closely matches what a user would expect
"I always thought the sorting here was done by calendar date, but I'm seeing instances were this is not the case. It's actually done by incident number.
I think that is not a very useful sorting, as it is not related to any data in the incident itself, even we can argue that it is loosely related to "date", it just represent the order in which they were entered.
This could be changed to the date of the event, or better yet (as a feature request) to implement sorting by different criteria (date, category, etc)
Cheers,
Pablo"
More of a feature request, sort reports using additional criteria apart from date/approval or verified status.
The text was updated successfully, but these errors were encountered: