http://localhost/ticket/2765 <p> I'd suggest we add a generic "failed to login, invalid fields" in the error_summary block. Then more specific "this field is required" when a required field is left empty. </p> <p> So in general, provide an error dict </p> en-us http://assets.okfn.org/p/ckan/img/ckan_logo_shortname.png http://localhost/ticket/2765 Trac 0.12.3 toby Tue, 31 Jul 2012 11:58:37 GMT http://localhost/ticket/2765#comment:1 http://localhost/ticket/2765#comment:1 <ul> <li><strong>owner</strong> changed from <em>toby</em> to <em>aron.carroll</em> </li> <li><strong>status</strong> changed from <em>new</em> to <em>assigned</em> </li> </ul> <p> @aron, </p> <p> I've added a data summary </p> <p> The data errors for empty fields are not happening due to our use of repoze.who for logins which is crap and doesn't provide feedback re loggin details so not possible without hacking repoze which i'm not doing. </p> <p> Could you look at cleaning up the error summary we don't supply a field name (key) as not relevant it would be nice if we could hide the : also removing the invalid fields message would be good to hide maybe add this as an option to the macro </p> Ticket aron.carroll Wed, 01 Aug 2012 10:44:18 GMT http://localhost/ticket/2765#comment:2 http://localhost/ticket/2765#comment:2 <ul> <li><strong>status</strong> changed from <em>assigned</em> to <em>closed</em> </li> <li><strong>resolution</strong> set to <em>fixed</em> </li> </ul> <p> Cool, closed as of 99b8c4f. I fixed the error summary display and highlighted the fields in red to show they were incorrect. </p> Ticket