search

Em-Ant / fcc-options-app

Options Inc. web app development
Other
12 stars 7 forks source link

Max passengers per vehicle constraint not being observed #24

Closed andrealonzo closed 8 years ago

andrealonzo commented 8 years ago

If the max passengers per vehicle is 14, you can add 15 passengers on a vehicle right now without an error.

Em-Ant commented 8 years ago

Can this be validated in the vehicles 'pre save' middleware ?

Em-Ant commented 8 years ago

This should fix this issue 028d2d6. Please, check it (and close if it's all ok). We still have to validate when lowering maximum Passengers on setting, with consumers on vehicles exceeding the new setting.

andrealonzo commented 8 years ago

Looks good. Closing this issue. I'm going to open a new issue when updating new settings.