diff options
| -rw-r--r-- | app/controllers/request_controller.rb | 8 | ||||
| -rw-r--r-- | public/stylesheets/main.css | 6 | ||||
| -rw-r--r-- | todo.txt | 15 |
3 files changed, 25 insertions, 4 deletions
diff --git a/app/controllers/request_controller.rb b/app/controllers/request_controller.rb index cfb9c1a05..f891c981a 100644 --- a/app/controllers/request_controller.rb +++ b/app/controllers/request_controller.rb @@ -4,7 +4,7 @@ # Copyright (c) 2007 UK Citizens Online Democracy. All rights reserved. # Email: francis@mysociety.org; WWW: http://www.mysociety.org/ # -# $Id: request_controller.rb,v 1.139 2008-11-10 18:08:29 francis Exp $ +# $Id: request_controller.rb,v 1.140 2008-11-11 13:59:26 francis Exp $ class RequestController < ApplicationController @@ -199,7 +199,11 @@ class RequestController < ApplicationController @info_request.save! # XXX send_message needs the database id, so we send after saving, which isn't ideal if the request broke here. @outgoing_message.send_message - flash[:notice] = "Your " + @info_request.law_used_full + " request has been created and sent on its way!" + flash[:notice] = "<p>Your " + @info_request.law_used_full + " request has been <strong>sent on its way</strong>!</p> + <p><strong>We will email you</strong> when there is a response, or after 20 working days if the authority still hasn't + replied by then.</p> + <p>If you write about this request (for example in a forum or a blog) please link to this page, and add an + annotation below telling people about your writing.</p>" redirect_to request_url(@info_request) else # do nothing - as "authenticated?" has done the redirect to signin page for us diff --git a/public/stylesheets/main.css b/public/stylesheets/main.css index 3cbaa4b7b..a863d2679 100644 --- a/public/stylesheets/main.css +++ b/public/stylesheets/main.css @@ -613,6 +613,12 @@ div.pagination { text-align: center; padding-top: 0.3em;} background-color: #D5FFD8; padding: 0.5em; } +/* This doesn't seem to work, not sure why (if you turn it on, check all + * flashes with <strong> in them work OK) +#notice * strong +{ + font-weight: bolder; +} */ .describe_state_form, .undescribed_requests, .gone_postal_help { font-weight: normal; margin-bottom: 1em; @@ -10,8 +10,16 @@ If you've already conducted an internal review, at all places don't offer it again. Clock for internal review -"The internal review should take 2-3 weeks for simple cases, and up to 6 weeks -even for complex reviews." + The Information Commissioner has issued a "Good Practice Guidance" document: + http://www.ico.gov.uk/upload/documents/library/freedom_of_information/detailed_specialist_guides/foi_good_practice_guidance_5.pdf + Reviews are meant to be carried out "as quickly as possible", and they + suggest 20 working days as a reasonable limit. In exceptional cases they + accept it may take longer, but recommend informing the requester in such + a case as to why. They do not believe it should ever take longer than 40 + days. +Fix up the text: "The internal review should take 2-3 weeks for simple cases, +and up to 6 weeks even for complex reviews." + Awaiting internal review overdue state? Search for text "internal review" in followups and add warning if they aren't @@ -54,6 +62,7 @@ CSS / design things - icons for "Things to do with this request" ? - Improve CSS on IE7 for large images in docs http://www.whatdotheyknow.com/request/3289/response/7810/attach/html/3/20081023ReplyLetter.pdf.html + - "fugly" (search email archive) in IE6/XP, Alex Skene can find it Write code to make sure the Return-Path is never foi@sandwich.org.uk (even if the Rails code breaks for Sendmail case in future botched Rails upgrades :) @@ -86,6 +95,8 @@ Add "I want to file squillions of requests" Add "I didn't get the email / spam question" and link to from FOI officer upload page Add "Postal replies stuff" and link to from FOI officer upload page Add "I'm not sure which authority to make my request to" - make flow better after first section, to abrupt now +I think the advice in this annotation could go into a nice comment: +http://www.whatdotheyknow.com/request/berr_response_to_eu_on_phorm_bt#comment-356 Link the overdue date to an explanation of what bank holidays it skipped etc. :) |