Eric
Posts: 17312
Joined: 10/10/2003 From: Seattle, WA Status: online
|
Whew, it has been a busy several days since my year-end email with dozens and dozens of bulk imports and almost a thousand emails to reply to (and still 50-60 more hanging in my inbox I plan to get to). I just checked in code to cleanup the process of accepting pending deliveries: - If editing a specific pending purchase from the wine detail sub-page, clicking ACCEPT PENDING will change the delivery date to today's date.
The bulk accept scenario is now much more robust:- When you click to submit the button changes to a progress animation, so it is not possible to double-submit.
- I now trap a number of error conditions such as an invalid delivery date or the failure to select any purchases, and these give appropriate feedback.
- Upon successful completion I was just refreshing the screen. Now, like the old site, I actually redirect you to a screen where you can bulk relocate bottles. In the FUTURE I intend to give you a few choices including closing the dialog and refreshing the screen of pending deliveries, bulk-relocating or eventually an individual bottle editable screen.
- HUMAN SPEAK: Please don't accept more than a few hundreds bottles for delivery at once, or you will hit a user interface glitch. GEEKSPEAK: Also for the future, right now if you deliver more than 400 or so bottles it will choke due to querystring length. In the future I plan to do some work to use querystring based lookups when the number of bottles is small enough. If/when you get towards the error condition I will switch the form from http GET to http POST to avoid the error. I tend to avoid POST, as you can't refresh it or save it as a favorite due, but it gets you around the 4096 character limitation of URL's.
I also did some small cleanup on the payment screen such as one of the first actual working help tips (they will be all over the editing forms and documented very helpful someday) and instructions on how to pay via PayPal.
_____________________________
Cheers! -Eric LeVine http://twitter.com/cellartracker http://facebook.com/cellartracker
|