darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-07-13 09:50:08 -04:00
Code Issues Packages Projects Releases Wiki Activity
5,839 Commits 59 Branches 61 Tags
dcc0eb1e5201dff97665051cdd39a71cde19ec9d
Commit Graph

5839 Commits

This Branch
This Branch
All Branches
Author SHA1 Message Date
Tom Byers
dcc0eb1e52 Change test on URL to use urllib methods 
Checking the raw string was breaking the test
sometimes due to it expecting the query parameters
to be in a different order.
 2018-08-08 16:24:52 +01:00
Tom Byers
155f6b9626 Merge pull request #2209 from alphagov/add-email-branding-preview 
Preview the email branding before setting it
 2018-08-08 15:25:25 +01:00
Chris Hill-Scott
bb4c77fd59 Merge pull request #2212 from alphagov/domains-update 
Added a few contract / MOU to the signed list
 2018-08-08 14:49:53 +01:00
Tom Byers
d931966e5a Add CSS for email preview iframe 2018-08-08 14:23:38 +01:00
Pete Herlihy
f9cacf606c Added a few contract / MOU to the signed list 2018-08-08 12:30:07 +01:00
Tom Byers
e7ead052ce Add tests for /_email page 2018-08-08 11:21:30 +01:00
Tom Byers
d8d48ea801 Add tests for preview step 
Includes moving code that tests an API call to
update the service with the new branding to the
preview step submission.

Also includes a change to the HTTP params sent by
the set-email-branding step. Think it was missed
out of this PR:

https://github.com/alphagov/notifications-admin/pull/1843

...so was never changed to 'branding_style'.
 2018-08-07 17:05:30 +01:00
Tom Byers
eb72f43b3f Fixes for tests broken by changes 
- corrects target page for set_email_branding to
  new preview step instead of itself
- removed check for helper method being called in
  email page test
- updates expected result for test of global
  headers to include changes to `frame-src`
- updates navigation config with brand preview page
 2018-08-07 16:56:37 +01:00
Tom Byers
386105954e Add JS to embed email preview into page 2018-08-07 16:56:37 +01:00
Tom Byers
c7610f359a improve range of params email page can take 
Can now deal with no query params and has a
default format of the GOV.UK branding.
 2018-08-07 16:56:37 +01:00
Tom Byers
a2a02a5185 Add preview step to branding selection flow 
Gives platform admins a chance to preview the
combination of brand type and custom brand
(coloured banner and logo) set for service before
saving.
 2018-08-07 16:56:37 +01:00
Tom Byers
7b955ffad2 Add 'self' to 'frame-src' header 
Allows iframes to contain pages from the same
domain as the parent page.
 2018-08-07 16:56:36 +01:00
Rebecca Law
6a06d4d714 Merge pull request #2171 from alphagov/flexible-data-retention 
Flexible data retention
 2018-08-07 14:27:31 +01:00
Chris Hill-Scott
c07a36b283 Merge pull request #2206 from alphagov/insolvency.gov.uk 
Mark agreement signed by The Insolvency Service
 2018-08-07 09:49:53 +01:00
Chris Hill-Scott
367dc05952 Mark agreement signed by The Insolvency Service 
Crown according to: http://www.nationalarchives.gov.uk/information-management/re-using-public-sector-information/uk-government-licensing-framework/crown-copyright/uk-crown-bodies/#azi

Signed agreement: https://drive.google.com/open?id=0ByP6zL-gKn64NEpHWEdxWnYySnJVZnkyd2RDMDByaEkwT1Nj
 2018-08-07 09:27:05 +01:00
Rebecca Law
18750585c4 Reduced the size of the pull request by only dealing with adding the data retention form. 2018-08-06 16:04:31 +01:00
Rebecca Law
ae1f1c95d6 Merge branch 'master' into flexible-data-retention 2018-08-06 15:37:11 +01:00
Rebecca Law
b0da7887a8 Merge branch 'master' of github.com:alphagov/notifications-admin 2018-08-06 15:32:14 +01:00
Chris Hill-Scott
4f7388f9b6 Merge pull request #2203 from alphagov/corby.gov.uk 
Mark agreement signed by Corby borough council
 2018-08-06 12:06:18 +01:00
Chris Hill-Scott
f4c96d5d54 Mark agreement signed by Corby borough council 2018-08-06 11:42:47 +01:00
Chris Hill-Scott
b93d5a2c6f Merge pull request #2200 from alphagov/pyup-update-pytest-3.7.0-to-3.7.1 
Update pytest to 3.7.1
 2018-08-03 16:11:55 +01:00
Chris Hill-Scott
776d54ed54 Merge pull request #2201 from alphagov/bump-utils-29.3.4 
Bump utils to 29.3.4
 2018-08-03 15:05:30 +01:00
Chris Hill-Scott
55054b81b5 Include test key data by default on platform admin 
This is the existing behaviour. It’s broken by this issue in WTForms
2.2.1: https://github.com/wtforms/wtforms/issues/401

This commit hand-crafts the default value, because WTForms is ignoring
the `default` argument on the form object attribute.

Not really sure how this ever worked 🤔
 2018-08-03 14:53:40 +01:00
Chris Hill-Scott
0d2de75e84 Bump utils to 29.3.4 
https://github.com/alphagov/notifications-utils/compare/29.3.2...bullet-for-bullet
 2018-08-03 13:57:55 +01:00
pyup-bot
8d95673349 Update pytest from 3.7.0 to 3.7.1 2018-08-02 20:47:56 +01:00
Chris Hill-Scott
dd2364654a Merge pull request #2166 from alphagov/basic-view-description 
Revise how we talk about what basic view is
 2018-08-02 17:41:26 +01:00
Chris Hill-Scott
26823b6122 Fix straight apostrophe 2018-08-02 17:35:37 +01:00
karlchillmaid
8eaf7caa05 Updated hint text 
Updated hint text
 2018-08-02 17:31:16 +01:00
Chris Hill-Scott
6d7c47111c Merge pull request #2198 from alphagov/basic-event-keys 
Fix broken keys for basic view event tracking
 2018-08-02 17:13:48 +01:00
karlchillmaid
48ddce8410 Removed reference to 'their' sent messages 
Removed reference to 'their' sent messages – because users will be able to see all sent messages
 2018-08-02 17:09:20 +01:00
karlchillmaid
5c5fe54cb4 Updated content to include next steps 
Updated content to explain what users need to do next.

Includes explicit instructions and a repetition of 'team members'.
 2018-08-02 16:58:43 +01:00
Chris Hill-Scott
cabb0a74c1 Fix broken keys for basic view event tracking 
Bad copy/paste…
 2018-08-02 16:44:15 +01:00
Chris Hill-Scott
d92340b7cf Merge pull request #2197 from alphagov/track-basic-view 
Track onboarding of basic view in Google Analytics
 2018-08-02 16:17:14 +01:00
Chris Hill-Scott
c655826b17 Merge pull request #2195 from alphagov/geo.gov.uk 
Add Government Equalities Office to domain list
 2018-08-02 15:24:30 +01:00
Chris Hill-Scott
fc5f3ee132 Merge pull request #2192 from alphagov/jobs-one-off 
Show uploaded files in basic view
 2018-08-02 15:23:58 +01:00
Chris Hill-Scott
8e6efadb23 Track virtual page view when enabling basic view 
So we can see what pages people go to after turning it on.
 2018-08-02 15:23:52 +01:00
Chris Hill-Scott
77d96032bf Track event when someone previews basic view 
We’re interested to know which services are looking at the basic view
preview (even if they’re not turning it on).

This commit logs their service IDs as events into Google Analytics.
 2018-08-02 15:23:52 +01:00
Chris Hill-Scott
291b919e78 Add Government Equalities Office to domain list 
They have been using Notify for the Gender Pay Gap service for ages.
 2018-08-01 12:54:36 +01:00
Chris Hill-Scott
c551ce9a42 Show jobs in basic view 
There are some teams who send jobs on a daily/weekly basis. They have
team members who only use Notify for this purpose. So they would
probably benefit from basic view, because they don’t need to see the
dashboard.

This commit:
- adds a new item (uploaded files) to the basic view navigation for
  teams that have sent at least one job
- makes the job pages visible to basic view users

I think we should do this now, rather than as a later enhancement to
basic view. We only have one chance to announce the feature, so teams
who do send jobs may otherwise discount it as not useful for them and
the opportunity to have them use it is lost.
 2018-08-01 10:45:02 +01:00
Rebecca Law
4f34025e7f Merge branch 'master' of github.com:alphagov/notifications-admin 2018-08-01 10:42:22 +01:00
Rebecca Law
505c8b1224 Merge pull request #2194 from alphagov/fix-template-duplication 
Edit, don’t duplicate, existing templates
 2018-07-31 16:31:56 +01:00
Rebecca Law
d512f330cd Edit, don’t duplicate, existing templates 2018-07-31 16:23:46 +01:00
Rebecca Law
4b32d9aa0c Edit, don’t duplicate, existing templates 2018-07-31 16:20:10 +01:00
Chris Hill-Scott
b35f6e9d5f Merge pull request #2180 from alphagov/service-model 
Make a service model
 2018-07-31 13:31:09 +01:00
karlchillmaid
6b33bc7956 Remove reference to 'one-off' messages 
Basic view should work for sending multiple messages too.
 2018-07-31 12:35:36 +01:00
Chris Hill-Scott
9d2e0ae2f3 Merge pull request #2191 from alphagov/json-boolean 
Store boolean values in Redis as valid JSON
 2018-07-31 12:24:38 +01:00
Chris Hill-Scott
45579c59cd Store boolean values in Redis as valid JSON 
Calling `.set()` with `True` stores the byte string `'True'` which
cannot subsequently be decoded from JSON (because boolean values in
JSON are lowercase, ie `true`).
 2018-07-31 12:17:28 +01:00
Chris Hill-Scott
1073b8f178 Merge pull request #2178 from alphagov/cache-has-jobs 
Check if any jobs exist before querying jobs
 2018-07-31 11:17:42 +01:00
Alexey Bezhan
5af1183218 Merge pull request #2153 from alphagov/pin-all-requirements 
Pin all requirements in requirements.txt
 2018-07-31 10:25:04 +01:00
Chris Hill-Scott
0906eb019d Check that API is not called when result is cached 
Good to explicitly test this (rather than relying on an exception not
being raised).
 2018-07-31 10:05:10 +01:00