I've overridden some of DefaultAdminController's methods and options configuration template to implement this, but I think we need more general approach for adding validators to tool's options in the future. Perhaps we can allow to specify validator(s) when defining ConfigOptions and implement actual validation in DefaultAdminController.configure + add ajax form submit/error handling in general template too, so then custom app can just specify validator and don't add any boilerplate. I didn't implement this now, because I think it is beyond the scope of this ticket, but maybe we should have this.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Closed #747.
ib/7820I've overridden some of
DefaultAdminController's methods and options configuration template to implement this, but I think we need more general approach for adding validators to tool's options in the future. Perhaps we can allow to specify validator(s) when definingConfigOptions and implement actual validation inDefaultAdminController.configure+ add ajax form submit/error handling in general template too, so then custom app can just specify validator and don't add any boilerplate. I didn't implement this now, because I think it is beyond the scope of this ticket, but maybe we should have this.I agree a more "formal" way to enforce validations would be good, but this is sufficient for now.