diff options
| -rw-r--r-- | app/helpers/link_to_helper.rb | 7 | ||||
| -rw-r--r-- | app/views/layouts/default.rhtml | 7 | ||||
| -rw-r--r-- | todo.txt | 5 |
3 files changed, 12 insertions, 7 deletions
diff --git a/app/helpers/link_to_helper.rb b/app/helpers/link_to_helper.rb index 7ef8a8186..41a462280 100644 --- a/app/helpers/link_to_helper.rb +++ b/app/helpers/link_to_helper.rb @@ -5,7 +5,7 @@ # Copyright (c) 2007 UK Citizens Online Democracy. All rights reserved. # Email: francis@mysociety.org; WWW: http://www.mysociety.org/ # -# $Id: link_to_helper.rb,v 1.6 2007-12-24 16:49:36 francis Exp $ +# $Id: link_to_helper.rb,v 1.7 2008-01-02 20:18:06 francis Exp $ module LinkToHelper @@ -30,8 +30,11 @@ module LinkToHelper link_to h(public_body.name), public_body_url(public_body) end + def user_url(user) + return show_user_url(:simple_name => simplify_url_part(user.name)) + end def user_link(user) - link_to h(user.name), show_user_url(:simple_name => simplify_url_part(user.name)) + link_to h(user.name), user_url(user) end def info_request_link(info_request) diff --git a/app/views/layouts/default.rhtml b/app/views/layouts/default.rhtml index 29629ae1b..ef468c522 100644 --- a/app/views/layouts/default.rhtml +++ b/app/views/layouts/default.rhtml @@ -14,9 +14,12 @@ <div id="tagline">It's your information. Free it from them.</div> </div> <ul id="navigation"> + <li><a href="/new">Make Request</a></li> <li><%= link_to "All Requests", request_list_url %></li> - <li><a href="/new">New Request</a></li> - <!-- <li><a href="/my">My Requests</a></li> --> + <% if @user %> + <li><%=link_to "My Requests", user_url(@user) %></li> + <% else %> + <% end %> <!-- <li><a href="/about">About</a></li> --> </ul> @@ -31,16 +31,15 @@ Either rotate log files, or merge with Apache ones Let requester send follow-ups - but to which email address???!! aargh Do something after 20 working days if you get no response + display the working days correctly Work out how to get it to tell you code coverage of .rhtml files Make it validate the HTML maybe with http://www.anodyne.ca/wp-content/uploads/2007/09/be_valid_xhtml.rb -Test that actually sending request outgoing mail using deliveries +Test that it is actually sending the request outgoing mail, by using deliveries Remove "Outgoing messages is invalid" error -Some way of getting to your own user page - Make it so "mysociety test" can be done on the servers, so it checks any packages are installed When you classify something, it prints the error message on the page after it for no reason |