[FEATURE]: An option to specify table type about resolved-bot HOT 4 CLOSED

Implemented an interface to the api layer to include a preference for tables. Table preference is decided after each loop of time searches. So the structure is basically, given a priority list of times {T1 T2} and a priority list of table types {Ta1 Ta2}, the checking goes(rough pseudocode)

For slot in slots {
If slot.Time == T1 && slot.Type == Ta1 -> reserve
}

For slot in slots {
If slot.Time == T2 && slot.Type == Ta1 -> reserve
}

For slot in slots {
If slot.Time == T1 && slot.Type == Ta2 -> reserve
}

For slot in slots {
If slot.Time == T2 && slot.Type == Ta2 -> reserve
}

The changes involved modifying the resy api reserve operation, specifically the find step. We search the config of the slot for a field called type, which provides a string declaration of the type of table the slot corresponds to. See api/resy/doc.go for more info.

We still need to implement the command line flag for adding table types, but @justin-s-wong already has a commit in progress for this, should be available sometime tomorrow.

from resolved-bot.

Ok, the command line parsing is done. I'd like some initial testing done for this. @ndbs879827 you can test these changes by cloning the branch 29-feature-an-option-to-specify-table-type and rebuilding with go build. Let us know if this is what you had in mind

from resolved-bot.

As an extra note, since these changes pass my initial testing, if you don't have any other concerns, this branch will get merged into develop in a day or so

from resolved-bot.

I've merged the changes into develop, but I'll keep the feature branch and this issue open for two days to anyone who would like to voice their opinion

from resolved-bot.