- **status**: in-progress --> review - **Comment**: Closed #747. `ib/7820`
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 `ConfigOption`s 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. --- ** [tickets:#7820] Validate URLs when configuring external link tool** **Status:** review **Milestone:** unreleased **Labels:** sf-1 bitesize 42cc sf-current ux **Created:** Mon Jan 26, 2015 05:14 PM UTC by Dave Brondsema **Last Updated:** Tue Mar 10, 2015 07:59 AM UTC **Owner:** Igor Bondarenko We should validate URLs when setting up an external link. And add a missing http(s):// automatically would be helpful. --- Sent from forge-allura.apache.org because [email protected] is subscribed to https://forge-allura.apache.org/p/allura/tickets/ To unsubscribe from further messages, a project admin can change settings at https://forge-allura.apache.org/p/allura/admin/tickets/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.
