Closed
Bug 947425
Opened 11 years ago
Closed 11 years ago
Default supersearch should not include a date field
Categories
(Socorro :: Webapp, task, P3)
Socorro
Webapp
Tracking
(Not tracked)
VERIFIED
FIXED
69
People
(Reporter: benjamin, Assigned: adrian)
Details
Attachments
(1 file)
250.21 KB,
image/png
|
Details |
When you click the initial search link, it is pre-populated with a "date" "<=" "the hour before now". However, as I understand it there is a default date range for searches over the last week.
When pasting links into bugzilla, we typically want them to be "dateless" links, so that people clicking them get the last week's worth of data rather than whenever the link was pasted. To make that easier, I think we should start out the search form without the pre-populated date field, *or* make it a relative time field of some sort e.g. "date" ">=" "one week ago"
Assignee | ||
Comment 1•11 years ago
|
||
As a FYI, the default date field was put here for two reasons: convenience for users (but it turns out it's not that convenient :) ) and small performance improvements in the back-end.
Removing it is really simple. Changing its default to be a range is not. I will thus go with the former solution for now.
Assignee: nobody → adrian
OS: Linux → All
Priority: -- → P3
Hardware: x86_64 → All
Assignee | ||
Comment 2•11 years ago
|
||
Pull request: https://github.com/mozilla/socorro/pull/1721
Comment 3•11 years ago
|
||
Commits pushed to master at https://github.com/mozilla/socorro
https://github.com/mozilla/socorro/commit/c24379785fe2d8bf63074e62ee65596ee5ba9c8f
Fixes bug 947425 - Removed the default date in Super Search.
https://github.com/mozilla/socorro/commit/b8d33d0d561046840685e30c3dca7fd447b074ab
Merge pull request #1721 from AdrianGaudebert/947425-remove-default-date-supersearch
Fixes bug 947425 - Removed the default date in Super Search.
Updated•11 years ago
|
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Comment 4•11 years ago
|
||
Verified FIXED on https://crash-stats.allizom.org/search/ -- default is removed.
Status: RESOLVED → VERIFIED
Comment 5•11 years ago
|
||
Updated•11 years ago
|
Target Milestone: --- → 69
You need to log in
before you can comment on or make changes to this bug.
Description
•