Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Orders list doesn't sort by delivery date correctly #3479

Closed
rbarreca opened this issue Oct 19, 2018 · 0 comments
Closed

Orders list doesn't sort by delivery date correctly #3479

rbarreca opened this issue Oct 19, 2018 · 0 comments
Projects
Project Board

Comments

@rbarreca
Copy link
Contributor

Describe the bug
If two deliveries have same supplier cutoff, the Orders list may not sort by delivery date correct

To Reproduce
Steps to reproduce the behavior:

  1. Go to Orders list for Farm Link
  2. Sort by delivery date
  3. Notice Sat and Fri dates might be jumbled together

Expected behavior
Should sort by the displayed date correct...and that should be buyer delivery date/time.

Screenshots
screen shot 2018-10-18 at 9 26 53 pm
@rbarreca rbarreca added this to Icebox in Product Backlog via automation Oct 19, 2018
@rbarreca rbarreca moved this from Icebox to Groomed product backlog in Product Backlog Oct 19, 2018
rbarreca pushed a commit that referenced this issue Nov 2, 2018
Fix delivery sorting on choose delivery page and orders list page (#3478
23aed3d 
, #3479)

- Sort by buyer_deliver_on instead of deliver_on (when supplier drops)
- Remove cruft.
@rbarreca rbarreca mentioned this issue Nov 2, 2018
Merged
@rbarreca rbarreca added this to Needs review in Project Board Nov 2, 2018
@rbarreca rbarreca removed this from Groomed product backlog in Product Backlog Nov 6, 2018
@rbarreca rbarreca moved this from Needs review to Awaiting QA in Project Board Nov 7, 2018
@rbarreca rbarreca moved this from Awaiting QA to QA Accepted in Project Board Nov 7, 2018
@rbarreca rbarreca moved this from QA Accepted to Released in Project Board Nov 7, 2018
@rbarreca rbarreca closed this as completed Nov 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
Project Board
  
Released
Milestone
No milestone
Development

No branches or pull requests
1 participant
@rbarreca