Date search allows entry of invalid dates

thegooglecodearchive / allforgood

Automatically exported from code.google.com/p/allforgood

0 stars 0 forks source link

Date search allows entry of invalid dates #570

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

Start and end date fields allow entry of end dates before the start.  
Start of 1/30/11 and end date of 1/1/11

Original issue reported on code.google.com by danstryk...@gmail.com on 9 Jan 2011 at 5:50

GoogleCodeExporter commented 9 years ago

Can also enter an end date while leaving the start date blank which will 
results in opps be returned that start after the end date.  See attached screen 
shot result A.

Original comment by danstryk...@gmail.com on 9 Jan 2011 at 6:08

Changed title: Date search allows entry of invalid dates
Added labels: Priority-High
Removed labels: Priority-Medium

Attachments:

570.jpg

GoogleCodeExporter commented 9 years ago

We have removed end date.

Original comment by ejst...@gmail.com on 9 Jan 2011 at 7:40

GoogleCodeExporter commented 9 years ago

Original comment by ejst...@gmail.com on 9 Jan 2011 at 7:53

Changed state: Fixed

GoogleCodeExporter commented 9 years ago

Will need to review how this logic works thoroughly.  It seems strange to only 
have start date and no end date from a UX perspective.

Original comment by danstryk...@gmail.com on 9 Jan 2011 at 2:16

GoogleCodeExporter commented 9 years ago

Showing "Filter By Date" and only having one unlabeled date field is not a good 
UX.  See first attached screen shot.  If we are going to support a date 
filtering feature that allows selection of a start date, we should also show a 
field for entering the end date.  If cannot get this logic may need to revert 
to using the approach currently in production.  See below and 2nd attached 
screen shot.
When
Anytime
Today
This weekend
This week
This month

Original comment by danstryk...@gmail.com on 9 Jan 2011 at 3:13

Changed state: Started
Added labels: Priority-Critical
Removed labels: Priority-High

Attachments:

[570-stage single date entry field.jpg](https://storage.googleapis.com/google-code-attachments/allforgood/issue-570/comment-5/570-stage single date entry field.jpg)
[570-production date range selection.jpg](https://storage.googleapis.com/google-code-attachments/allforgood/issue-570/comment-5/570-production date range selection.jpg)

GoogleCodeExporter commented 9 years ago

Original comment by jwdemp...@gmail.com on 9 Jan 2011 at 3:56

GoogleCodeExporter commented 9 years ago

Please retest

Original comment by ejst...@gmail.com on 9 Jan 2011 at 11:09

Changed state: Fixed

GoogleCodeExporter commented 9 years ago

fix works when using the date pickers.  invalid dates can still be typed in 
directly into the date fields though.  not an issue that needs to be addressed 
though.

Original comment by danstryk...@gmail.com on 10 Jan 2011 at 5:47

Changed state: Verified