aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--app/views/comment/new.rhtml10
-rw-r--r--todo.txt2
2 files changed, 7 insertions, 5 deletions
diff --git a/app/views/comment/new.rhtml b/app/views/comment/new.rhtml
index db73d247f..c0e7e3a22 100644
--- a/app/views/comment/new.rhtml
+++ b/app/views/comment/new.rhtml
@@ -23,9 +23,6 @@ Annotations are so anyone, including you, can help the requester with their requ
<li> Advise on how to <strong>best clarify</strong> the request.</li>
<% end %>
-<% if @info_request.awaiting_description %>
-<% end %>
-
<% if not [ 'successful', 'partially_successful' ].include?(@info_request.described_state) %>
<li> Link to the information requested, if it is <strong>already available</strong> on the Internet. </li>
<li> Suggest <strong>where else</strong> the requester might find the information. </li>
@@ -44,6 +41,10 @@ Annotations are so anyone, including you, can help the requester with their requ
<li> Point to <strong>related information</strong>, campaigns or forums which may be useful. </li>
<% end %>
+<% if [ 'gone_postal' ].include?(@info_request.described_state) %>
+ <li> A <strong>summary</strong> of the response if you have received it by post. </li>
+<% end %>
+
<% if [ 'not_held' ].include?(@info_request.described_state) %>
<li> Ideas on what <strong>other documents to request</strong> which the authority may hold. </li>
<% end %>
@@ -51,6 +52,9 @@ Annotations are so anyone, including you, can help the requester with their requ
<li> Advise on whether the <strong>rejection is legal</strong>, and how to complain about it if not. </li>
<% end %>
+<% if [ 'internal_review' ].include?(@info_request.described_state) %>
+ <li> <strong>Advice</strong> on how to get a response that will satisfy the requester. </li>
+<% end %>
<% if [ 'error_message' ].include?(@info_request.described_state) %>
<li> You know what caused the error, and can <strong>suggest a solution</strong>, such as a working email address. </li>
<% end %>
diff --git a/todo.txt b/todo.txt
index 682a07842..f4235df84 100644
--- a/todo.txt
+++ b/todo.txt
@@ -184,8 +184,6 @@ I have several email alerts set up. Is there any chance they could include part
(or, preferably, all) of the search criterion in the Subject: line? :o)
(Perhaps do it in the case when only one search criterion makes the mail)
-Update annotation help text for new states like internal_review and gone_postal ./views/comment/new.rhtml
-
Test data dumper that removes sensitive data, but lets trusted people play with
whole database on their own machine without risk of compromise (for Tony)
- can avoid rebuilding emails, attachments etc. sanitized provided we don't