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.

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

Default Time Window Logic #520

Open
vcardillo opened this issue Mar 9, 2016 · 1 comment
Open

Default Time Window Logic #520

vcardillo opened this issue Mar 9, 2016 · 1 comment

Comments

@vcardillo
Copy link
Member

We're having an issue where all customers are just selecting their delivery window to be 11-12, since (we surmise) that's what the app is defaulting to. In order to be able to use less drivers for more deliveries, we need to be able to smooth this out. Here are the simple changes I'd like us to make:

Spec 1

I'm adding DefaultTimeMode to the AvailableServices->OrderAhead->availableMenus->menus object:

menus": [
    {
        "Menu": {},
        "MenuItems": [],
        "Times": [],
        "SelectedTime": "random"
...

Logic
Possible values: random | useDefault | first

IF random, THEN select a random time window be the default selection
IF useDefault, then execute as per Spec 2 below
IF first, then this is the current behavior, where the first time window is the default selection

I've set the backend to use the random option, currently.

Spec 2

I'm adding an isDefault attribute to the AvailableServices->OrderAhead->availableMenus->menus->Times object:

{
    "start": "17:00",
    "end": "18:00",
    "available": true,
    "delivery_price": "1.99",
    "isDefault": false
},

Logic
IF isDefault is true, THEN set this time as the default time window for this particular menu

On dev.

@vcardillo vcardillo added this to the 2.69 [current] milestone Mar 9, 2016
@josephlausf josephlausf modified the milestones: 2.69 [current], 2.7 [next] Mar 9, 2016
@josephlausf
Copy link
Contributor

random window - http://recordit.co/Pc7bhBc0tt

@vcardillo vcardillo added the spec label Mar 30, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants