Commit Graph

4084 Commits

Author SHA1 Message Date
Chris Hill-Scott
08b665b2bd Mark agreement signed by Medway Council 2018-10-01 15:10:42 +01:00
Leo Hemsted
5179bdbdef Merge pull request #2338 from alphagov/expected-letter-rate
1st class expected letter delivery
2018-10-01 14:20:36 +01:00
Chris Hill-Scott
351a3dc4f3 Merge pull request #2314 from alphagov/validate-rtgl
Don’t trigger HTML5 validation on any form
2018-10-01 14:18:16 +01:00
Chris Hill-Scott
abac89ca2f Merge pull request #2329 from alphagov/remove-table-options
Remove redundant variable definition
2018-10-01 11:28:42 +01:00
Leo Hemsted
432f1286fe update notifications page to take into account first class! 2018-10-01 11:07:35 +01:00
Leo Hemsted
efab189ae8 update jobs to take into account first class 2018-10-01 11:07:33 +01:00
Leo Hemsted
32f39475d2 remove dupe of get_letter_timings
it was moved from admin to utils a while ago
2018-09-28 15:02:32 +01:00
Chris Hill-Scott
d34fed7612 Add domain for Greenwich council
Not sure why they weren’t in here…
2018-09-28 09:29:01 +01:00
Rebecca Law
0591d3e667 Merge pull request #2334 from alphagov/fix-download-bug
Fix download csv to use the service data retention if it exists.
2018-09-27 16:07:17 +01:00
Rebecca Law
7b9df8de29 Fix download csv to use the service data retention if it exists. 2018-09-27 14:51:36 +01:00
Chris Hill-Scott
6de0374492 Merge pull request #2327 from alphagov/add-mou-to-checklist
Add line about the data sharing and financial agreement to the checklist
2018-09-26 16:33:13 +01:00
Chris Hill-Scott
c251c1326e Remove ‘Download the’ call to action
Users come to this page from various places, including the new 'Sign
the…' link on the request to go live page.

Of these users:
- some won't have signed it
- some will have signed it
- some will see that it's complete and wonder why, as they haven’t
  actually done anything

So it’s more appropriate for the title of this page to be descriptive,
rather than an action.
2018-09-26 16:14:50 +01:00
Chris Hill-Scott
2fe1a60bbf Mark agreement signed by DWP 2018-09-26 11:54:02 +01:00
Chris Hill-Scott
3fa877bdd3 Make wording around agreement more straightforward
We think that the next page explains the nuance around service vs
organisation sufficiently well.
2018-09-26 10:37:30 +01:00
Chris Hill-Scott
901b924e35 Remove redundant variable definition
This unused variable is a hangover from when the permissions
were displayed horizontally in a table.
2018-09-25 17:00:54 +01:00
Pea (Malgorzata Tyczynska)
6d32d8b756 Merge pull request #2324 from alphagov/state_postage
Display postage on letter notification page
2018-09-25 13:36:34 +01:00
Pea Tyczynska
d982385042 Change wording following content review 2018-09-25 11:05:32 +01:00
Chris Hill-Scott
5dd45da08c Add line about MOU to checklist
2/3 of our incomplete requests to go live are incomplete because the
Data Sharing and Financial Agreement isn’t signed.

We reckon we can be pushier about this by saying it’s ‘incomplete’ where
we know the agreement is signed.

Where the agreement is signed we should confirm this, rather than make
the line disappear. This is so it makes more sense to someone who sees
it as ‘incomplete’, signs it, then comes back to the page.

If we don’t know whether or not the agreement is signed we should wait
until someone has got in touch with us by requesting to go live to
figure it out. So that’s why we’re not showing that line at all.
2018-09-25 10:33:50 +01:00
Chris Hill-Scott
c2ef522986 Make task list read better for screen readers
This duplicates how the task list pattern is coded in the GOV.UK
Prototype kit[1]. It adds ARIA attributes and the use of a
semantically-meaningful element (`<strong>`) to give more information to
screen reader users.

1. https://govuk-prototype-kit.herokuapp.com/docs/templates/task-list
2018-09-25 10:31:40 +01:00
Chris Hill-Scott
21d29a3090 Fix missing ‘complete’ tag in go live request
This tag was not showing up in the call to the Zendesk API because the
return value of a generator is not included as a member of that
generator (on things `yield`ed from it are).
2018-09-24 17:47:56 +01:00
Chris Hill-Scott
8e48b17a6e Merge pull request #2321 from alphagov/auto-tag-request-to-go-live
Tag request to go live tickets automatically
2018-09-24 17:13:52 +01:00
Chris Hill-Scott
34968fb5da Mark agreement signed by Teignbridge council 2018-09-24 16:26:17 +01:00
Chris Hill-Scott
8bb23e09f2 Refactor to reduce nesting and repetition 2018-09-24 14:52:32 +01:00
Chris Hill-Scott
f29cfc0d48 Auto-tag email branding requests 2018-09-24 14:52:31 +01:00
Chris Hill-Scott
de65c30624 Tag request to go live tickets automatically
At the moment we manually tag tickets as they come in so we can analyse
how many of each type we’re getting.

Further, we manually tag all the request to go live tickets once a month
to analyse how many are complete/incomplete.

All this tagging is useful, but quite time consuming. Notify already
knows this information and – using the Zendesk API – we can tag them
automatically.

I’ve checked with Holly and this is the taxonomy we want to use.
2018-09-24 14:52:31 +01:00
Chris Hill-Scott
f29c6c90c0 Add info about checklist to ticket
So that someone picking up a ticket can be warned that they should be
checking these things.
2018-09-24 14:52:31 +01:00
Chris Hill-Scott
45b1b11abb Refactor go-live readyness into service model
All this info is info about the service. So it’s better to put it on
the service model because:
- encapsulation
- later reuse
2018-09-24 14:52:30 +01:00
Pea Tyczynska
64f356d7b0 Display postage on letter notification page 2018-09-24 11:29:33 +01:00
Katie Smith
fee47aa348 Hide Postage setting when letters are not enabled for service
We don't show the sender addresses and letter branding on the settings page
if a service doesn't have letters enabled, so we should also hide the
postage for services that can't send letters.

(Postage can only be seen by Platform Admin users at the moment, but this will
change later).
2018-09-20 15:46:32 +01:00
Chris Hill-Scott
ae2f6f14fa Mark agreement signed by Kingston Council 2018-09-20 11:20:46 +01:00
Katie Smith
7aedc3a91d Merge pull request #2311 from alphagov/show-postage-class
Add letter class row to settings page, visible to platform admin only
2018-09-20 09:52:24 +01:00
Katie Smith
3815a04a6f Merge pull request #2308 from alphagov/quis-patch-1
Remove unused variable
2018-09-20 09:52:18 +01:00
Chris Hill-Scott
d011742593 Add owner to DBS domain
All domains should have an owner
2018-09-20 09:45:20 +01:00
Chris Hill-Scott
4ce82be8dd Merge pull request #2317 from alphagov/merthyr.gov.uk
Mark agreement signed by Merthyr Tydfil
2018-09-19 16:57:35 +01:00
Chris Hill-Scott
2b8da4a2fa Merge pull request #2315 from alphagov/3-becomes-4
Set min text message sender length to 4 characters
2018-09-19 16:57:27 +01:00
Katie Smith
2726467fc7 Minor quote and heading fixes 2018-09-19 16:53:31 +01:00
Chris Hill-Scott
2f298783e0 Mark agreement signed by Merthyr Tydfil 2018-09-19 16:45:05 +01:00
Chris Hill-Scott
36eac9d031 Set min text message sender length to 4 characters
We’ve learned of a change implemented today by the UK mobile
network operators, to stop allowing text message sender names
of 3 or less characters.

Adding this validation will not affect existing senders, only those
users trying to add to or update their senders.
2018-09-19 16:32:34 +01:00
Chris Hill-Scott
8504864015 Mark agreement signed by Warwickshire County Council 2018-09-19 16:24:09 +01:00
Chris Hill-Scott
d9da219b7e Use a macro for form tags
This will stop us repeatedly forgetting to add `novalidate` and
`autocomplete='off'` to our forms (which is how most of them are set
up).

It uses sensible defaults, based on how we most-commonly configure
forms:
- most of our forms are `post`ed (but this can be overridden)
- `autocomplete` should only be enabled where it makes sense, otherwise
  it’s more annoying than useful (but this can be overriden)
- we should never be using HTML5 form validation because our own error
  styles and messages are better
2018-09-19 12:43:15 +01:00
Chris Hill-Scott
5ab876aeab Don’t trigger HTML5 validation on go live request
We should probably have a reusable thing for this…
2018-09-19 11:34:20 +01:00
Katie Smith
9660f372f3 Change naming in code to use 'postage'
Renamed everything that was previously called 'letter class' or 'postage
class' to 'postage', which is the new name we have decided to use.
2018-09-19 09:20:09 +01:00
karlchillmaid
eb9768c91b Removed heading above radios
Removed heading above radios
2018-09-18 11:14:11 +01:00
karlchillmaid
02ad9ed0a3 Updated postage labels
Updated postage labels
2018-09-18 11:13:32 +01:00
karlchillmaid
a2a8496d52 Updated postage radio labels
Updated postage radio labels
2018-09-18 11:12:12 +01:00
karlchillmaid
c125e24432 Updated postage content
Updated postage content
2018-09-18 11:05:33 +01:00
Katie Smith
de9759b99a Add letter class row to settings page, visible to platform admin
Added a new row to the settings table, 'Post class', which shows the
default letter class of a service and is only visible to Platform Admin.

Also added a new page to enable Platform Admin users to change the
default letter class for a service - this only has two options at the
moment, 1st class only and 2nd class only.
2018-09-18 10:31:01 +01:00
Chris Hill-Scott
75be9e13af Mark agreement signed by Newcastle-under-Lyme 2018-09-17 11:50:51 +01:00
Chris Hill-Scott
5adacdab39 Remove unused variable 2018-09-14 14:27:29 +01:00
Chris Hill-Scott
8f2b51b428 Mark agreement signed by Ryedale council 2018-09-14 11:51:45 +01:00