Commit Graph

4307 Commits

Author SHA1 Message Date
Chris Hill-Scott
6ee62fa9be Merge pull request #1531 from alphagov/october-roadmap
Updated roadmap for October 2017 onwards
2017-10-05 15:11:30 +01:00
Chris Hill-Scott
580da8b6dd Merge pull request #1532 from alphagov/info-risk-mgmt
Info risk mgmt page added.
2017-10-05 15:10:47 +01:00
Chris Hill-Scott
5a15871b88 Dashes not underscores 2017-10-05 15:00:18 +01:00
Pete Herlihy
c50989c706 Tidied up for code style 2017-10-05 14:58:25 +01:00
Pete Herlihy
27cbd699c9 Tidied up code style 2017-10-05 14:47:34 +01:00
Pete Herlihy
d418075dab Added page with the information risk management approach 2017-10-05 14:39:57 +01:00
Pete Herlihy
2fab0b4582 Added test for the route to information_risk_management page 2017-10-05 14:25:20 +01:00
Pete Herlihy
18bacab039 Added route to information_risk_management page 2017-10-05 14:24:12 +01:00
Pete Herlihy
a3f3689c31 Updated roadmap for October 2017 onwards 2017-10-05 13:58:05 +01:00
Chris Hill-Scott
0aa8c412af Merge pull request #1529 from alphagov/pyup-update-pytest-3.2.2-to-3.2.3
Update pytest to 3.2.3
2017-10-05 10:25:26 +01:00
pyup-bot
a0c7eda75a Update pytest from 3.2.2 to 3.2.3 2017-10-04 22:16:59 +01:00
Chris Hill-Scott
f7211b164f Merge pull request #1528 from alphagov/cf-off
Remove cffi
2017-10-04 11:01:21 +01:00
Chris Hill-Scott
6f2e1e1c87 Remove cffi
It doesn’t seem to be used; tests pass without it being installed.

Was added in 10950bb8a6 with no explanation as to why it’s needed…
2017-10-04 10:39:57 +01:00
Chris Hill-Scott
676691646c Merge pull request #1527 from alphagov/pyup-update-cffi-1.11.0-to-1.11.1
Update cffi to 1.11.1
2017-10-04 10:15:08 +01:00
Chris Hill-Scott
61ce4c5a02 Merge pull request #1525 from alphagov/only-mandate-mandatory-columns
Only mandate mandatory address columns
2017-10-04 10:14:39 +01:00
Chris Hill-Scott
775fa1ac2d Merge pull request #1524 from alphagov/key-types-and-letters
Add note about relationship between team API key and letters
2017-10-04 10:14:32 +01:00
pyup-bot
9f554f1e4c Update cffi from 1.11.0 to 1.11.1 2017-10-04 08:32:56 +01:00
Chris Hill-Scott
76bee708af Merge pull request #1523 from alphagov/79-84
79 - 84
2017-10-03 14:13:52 +01:00
Chris Hill-Scott
95f6257852 Merge pull request #1512 from alphagov/email-reply-tweaks
Tweak email reply to pages, add IDs
2017-10-03 14:12:56 +01:00
Chris Hill-Scott
4154dbba1f Only mandate mandatory address columns
If you miss ‘postcode’ from your file then you get told that you need
‘address_line_1’, ‘address_line_2’, ‘address_line_3’, etc.

This is incorrect – the only required address columns are lines 1 and 2,
plus the postcode. So this commit corrects the error message to be
factually accurate.

We had a user report this to Fajer as a bug.
2017-10-03 14:08:25 +01:00
Chris Hill-Scott
1b6d9d7f2a Reformat whitespace to be more compact 2017-10-03 13:33:37 +01:00
Chris Hill-Scott
d7fc7b082f Make trial mode message less verbose
Frontloads the ‘not’ part of the message, and makes it shorter, so
it’s more likely to be read and understood. Also makes it fit better
with the new ‘Can’t be used to send letters’ message.
2017-10-03 13:32:10 +01:00
Chris Hill-Scott
9c25b2d2da Add note about letters
When trying to send letters using the API, the ‘team and whitelist’ key
is confusing. We don’t have addresses for your team members, nor is
there a whitelist for letter addresses. The actual behaviour is that
you’ll get an error if you try to use this key to send letters.

So, for services who have letters available, we should add a hint
telling users that team and whitelist is probably not the key they’re
looking for.
2017-10-03 13:29:12 +01:00
Chris Hill-Scott
f93a0c074b Test existing functionality
When picking an API key you get a choice of different key types. Which
types are available depends on whether or not your service is in trial
mode.

This commit adds missing tests for the existing functionality which
determined the types of key that should be available to the user.
2017-10-03 13:27:49 +01:00
Pete Herlihy
cdacdc388a 79 - 84
Pembrokeshire County Council
G Cloud Team
Government Whips' Office
DCLG Housing and Planning Data Collection Team
Returner team - Government Equalities Office
2017-10-03 13:04:17 +01:00
Chris Waszczuk
ccd24f516d Merge pull request #1522 from gov-cjwaszczuk/master
Updated runtime python version
2017-10-03 11:17:50 +01:00
chrisw
44f4197180 Updated runtime python version 2017-10-03 10:57:07 +01:00
Chris Waszczuk
c2a50a7c37 Merge pull request #1520 from gov-cjwaszczuk/master
Added letters info to service / admin dashboard & activity page
2017-10-03 10:40:59 +01:00
chrisw
43395bc9d9 Added letters info to service / admin dashboard & activity page 2017-10-03 10:28:34 +01:00
Venus Bailey
4434b4250b Merge pull request #1521 from alphagov/vb-platform-admin-admin
(reverse #1506) admin to api not passing trial_mode_service para
2017-10-02 17:27:57 +01:00
venusbb
33a2909a76 reverse code not seperating trial_mode_services 2017-10-02 14:26:08 +01:00
Chris Hill-Scott
7b12b7e03d Merge pull request #1513 from alphagov/add-letters-to-go-live
Add letters to the request to go live page
2017-10-02 10:53:08 +01:00
Leo Hemsted
cf3f4d5cee Merge pull request #1516 from alphagov/empty-inbound-sms
make sure inbound sms page doesn't break if you receive a blank inbou…
2017-09-29 13:51:30 +01:00
Leo Hemsted
bb44479b3e disable statsd when testing
or it won't run if you're offline
2017-09-29 10:16:20 +01:00
Leo Hemsted
57b4971ac1 update conftest and tests to contain accurate test data
specifically, we identify inbound vs outbound by presence of
"notify_number", so ensure that test content actually contains that
field
2017-09-29 10:15:17 +01:00
Chris Hill-Scott
77da7543c0 Merge pull request #1519 from alphagov/acas.org.uk
Adding Acas.org.uk to whitelist
2017-09-29 09:48:30 +01:00
Pete Herlihy
74992da1c8 Adding acas.org.uk to whitelist 2017-09-29 09:24:07 +01:00
Chris Hill-Scott
cc0e308076 Merge pull request #1518 from alphagov/quis-patch-1
Update email-message.scss
2017-09-28 16:15:58 +01:00
Chris Hill-Scott
26d1bda43c Update email-message.scss
Got my first and last mixed up…
2017-09-28 16:15:48 +01:00
Chris Hill-Scott
84a225ffdb Merge pull request #1517 from alphagov/right-padding-email-message
Fix overridden padding on email subject
2017-09-28 16:03:55 +01:00
Leo Hemsted
b1e809e82a make sure inbound sms page doesn't break if you receive a blank inbound message 2017-09-28 15:37:18 +01:00
Chris Hill-Scott
c35088796a Hide ID when only one reply to address is shown
Most user will only have one reply to address. Which means they should
never have to worry about IDs. And if you only have one then you never
need its ID, because the last remaining address will always be the
default.

So IDs should only be shown when a service has created more than one
reply to address.

This required a bit of visual tweaking of the _user list_ pattern,
because its spacing wasn’t defined in a way that worked when only the
name of the thing, and not its details were shown on the page.
2017-09-28 11:35:52 +01:00
Imdad Ahad
a273cfe951 Merge pull request #1511 from alphagov/imdad-katie-feat-make-add-letter-contact-validation
Make the letter contact block (service settings form) compulsory
2017-09-28 11:23:56 +01:00
Chris Hill-Scott
06caccdd26 Fix overridden padding on email subject
Something in a new version of GOV.UK Elements, Template, or Frontend
Toolkit has introduced a rules which removes padding for the last
column in a table.

This is undesirable in the case of email message previews.
2017-09-28 10:47:33 +01:00
Chris Hill-Scott
c78fac911a Say manage if there’s more than none
‘Change’ as a label for the link is misleading, because this is also the
page you go to in order to get the ID of a given reply to address.
‘Manage’ feels a bit more general.
2017-09-28 10:42:05 +01:00
Chris Hill-Scott
b68784207b Add email reply address to ID to the page
The first users of multiple email reply to addresses will be using the
API. This means that the need to be able to specify the ID of the reply
to address they want.

We chose to implement it like this instead of by passing the address in
directly because that means deploying code. For some teams deploying
code can take weeks, and we’d like to let teams have the flexibility to
make changes faster than this.

Same as for templates, you shouldn’t have to go to the _edit_ page in
order to get the ID. This means listing them on the page where you see
all the reply to addresses.

Listing the IDs like this means that it’s not really a table any more,
because the information isn’t organised in columns. So I think it makes
sense to reuse the pattern from the manage team page, which has a
similar relationship between the information.
2017-09-28 10:42:04 +01:00
Chris Hill-Scott
47ebb6190a Make language consistent
The `is_default` text and text of the checkbox should use the same
terminology (‘email address’ not ‘address’).
2017-09-28 10:42:04 +01:00
Chris Hill-Scott
8d4418cf61 Equalise spacing
Make the amount of space above and below is equal, and consistent with
the spacing when we show the checkbox.
2017-09-28 10:42:04 +01:00
Chris Hill-Scott
e65e98a9f1 Fix indentation 2017-09-28 10:42:03 +01:00
Chris Hill-Scott
8fb55ffa78 Grey out None when email reply to not set
This is consistent with how we differentiate other ‘unset’ values on the
settings page.
2017-09-28 10:42:03 +01:00