darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-05-27 09:29:22 -04:00
Code Issues Packages Projects Releases Wiki Activity
2,847 Commits 57 Branches 59 Tags
3a637c8fa98254cfbbf531341ff41c65771e50bf
Commit Graph

2847 Commits

This Branch
This Branch
All Branches
Author SHA1 Message Date
Jenny Duckett
3a637c8fa9 Move "Today" heading to left column on platform admin page 
When we make the numbers on this page more filterable the date range will be
one of the options to change, so it makes sense to move it to the side now
instead of leaving it above the big numbers.
 2016-12-06 12:42:27 +00:00
Jenny Duckett
c88a961e04 Add option to exclude test key stats on platform admin page 
This page currently includes all notifications for all services, including
those sent using a test key. Stats on all other pages exclude test key usage,
though, which can lead to confusion for admins comparing numbers between
pages. Adding the option to toggle between including and excluding test key
usage on the platform admin page gives context for this difference, and allows
admins to see live usage of the platform as well as load associated with test
key usage.
 2016-12-06 12:42:27 +00:00
Chris Hill-Scott
fc2ea8aeb8 Merge pull request #1044 from alphagov/fix-alignment-usage 
Fix alignment of year label on usage
 2016-12-06 10:23:13 +01:00
Chris Hill-Scott
b3cb3a761a Merge pull request #1043 from alphagov/column-fix 
Fix misaligned columns in CSV files
 2016-12-06 10:23:08 +01:00
bandesz
f43ba1fbb9 Enable SSE for S3 upload 2016-12-05 16:53:15 +00:00
minglis
18b4ce919b Merge pull request #1041 from alphagov/add-statsd 
Add statsd
 2016-12-05 15:59:06 +00:00
Chris Hill-Scott
62cb7b5d53 Fix alignment of year label on usage 
It got pushed down too far when we added the switch service bar in
https://github.com/alphagov/notifications-admin/pull/1029

This commit fixes it so that the text aligns with the baseline of the
`<h1>`.
 2016-12-05 13:02:19 +00:00
Chris Hill-Scott
18d11aa013 Move code for rendering messages/templates → utils 
Utils is better structured to handle the logic of what thing to show
for what template type, especially now that what we show for different
template types in different contexts has diverged significantly.

See https://github.com/alphagov/notifications-utils/commit/6b39c1a for
an example of this code moving into utils

Depends on and implements:
https://github.com/alphagov/notifications-utils/pull/84

The main reason for doing this is to get Paul’s fix for the misaligned
CSV columns: https://github.com/alphagov/notifications-utils/pull/87
 2016-12-05 12:11:54 +00:00
Andras Ferencz-Szabo
40f30ca13a Merge pull request #1042 from alphagov/docker_build_http_proxy 
Set http proxy params for Docker build
 2016-12-02 15:06:52 +00:00
bandesz
4c89c8c33d Set http proxy params for Docker build 2016-12-02 14:45:01 +00:00
minglis
68dc9f1966 Merge pull request #1038 from alphagov/generate-version-file-in-bootstrap 
Generate the version file in the bootstrap script
 2016-12-02 10:15:07 +00:00
Martyn Inglis
eab69fd840 Statsd disabled on dev machines 2016-12-01 17:22:05 +00:00
Martyn Inglis
0bc48897f3 Adds statsd to the application 
- statsd client from utils
- adds to logger an get same method based reporting as we do for the API.
 2016-12-01 17:21:06 +00:00
minglis
20d9964f71 Merge pull request #1040 from alphagov/request-id-logging 
Request id logging
 2016-12-01 13:31:58 +00:00
Martyn Inglis
c7549d552e Bumped version of utils 2016-12-01 13:13:48 +00:00
Martyn Inglis
0c9d188ef5 Bumped utils version 2016-12-01 11:34:18 +00:00
Martyn Inglis
08dc8fb13d Use the local APIClient rather than the one from the python-api-client 
- ensures that all API calls set the request ID when talking to the API.
 2016-11-30 17:01:44 +00:00
Martyn Inglis
655e6892ad Bumps client version 2016-11-30 17:01:02 +00:00
Martyn Inglis
202dd7d314 Override the BaseAPI Client 
- This allows us to set a custom header for admin calls only (not needed in client calls)
- Adds request-id from Middleware to the API call to ensure the API logs against the same request ID.
 2016-11-30 17:00:42 +00:00
Martyn Inglis
1384d2a61c Adds the request-id middleware to the admin app 
- adds a request-id to each request and passes that through to the response
- this is picked up by the loggers and used to tie log events to a request
 2016-11-30 16:59:47 +00:00
Andras Ferencz-Szabo
e412621a6e Merge pull request #1039 from alphagov/use_http_proxy_for_docker 
Use http proxy for Docker
 2016-11-30 16:54:53 +00:00
bandesz
b7d1a10e2d Use http proxy for Docker 2016-11-30 15:57:20 +00:00
Chris Hill-Scott
4161c152ab Merge pull request #1036 from alphagov/header-bar-colours 
Remove the red header meaning signed in to admin
 2016-11-29 18:11:41 +01:00
Jenny Duckett
5eca69882a Generate the version file in the bootstrap script 
This file needs to exist before the app can run, so create it automatically
rather than including it as an extra setup step in the README. The API app
already does this.
 2016-11-29 16:25:50 +00:00
Chris Hill-Scott
d9d24e3c60 Remove the red header meaning signed in to admin 
> Make the header bar red
>
> Red for admin is a good reckon.

– 286fc308d9 (part of https://github.com/alphagov/notifications-admin/pull/130)

Starting to think it’s not such a good reckon. Users could take a guess
at what it meant, but they often guessed wrong.

However, changing the colour of the header bar _is_ useful for us
internally to see which environment we’re in. So this commit makes three
changes:

1. On live, the header bar is always standard GOV.UK blue

2. On other environments, the header bar is some other colour (local is
   very different, staging and preview are related colours)

3. If an enviroment has a different header colour, it has it even when
   you’re not logged in.
 2016-11-25 17:14:53 +00:00
bandesz
82d6154a5f Add wheelhouse to .gitignore 2016-11-25 16:53:51 +00:00
Chris Hill-Scott
a8c68462b0 Merge pull request #1033 from alphagov/pc-need-new-npm 
Add npm (v3 or greater) as a README dependency
 2016-11-24 17:36:06 +01:00
Rebecca Law
1efa5530d4 Merge pull request #1027 from alphagov/deactivate-services-all-services-list 
dont show inactive services on the choose service list
 2016-11-23 16:10:44 +00:00
Rebecca Law
e0cecc7f83 Merge pull request #1034 from alphagov/upgrade-python-client 
Update python client to version 3.0.0 from 1.3.0
 2016-11-23 16:05:24 +00:00
bandesz
2734f27369 Add python-dev, libffi-dev, libssl-dev to the Docker build image 2016-11-23 12:41:07 +00:00
bandesz
a2b8e91ae7 Exclude cache directory from pep8 2016-11-23 12:40:39 +00:00
bandesz
0c00f3aae8 Add AGS elb to AWS scripts 2016-11-22 16:19:51 +00:00
Rebecca Law
d2f943ed2e Update python client to version 3.0.0 from 1.3.0 2016-11-22 15:04:40 +00:00
bandesz
b512c7980b Add variables to Makefile for codedeploy app name and s3 prefix 2016-11-22 12:24:26 +00:00
Paul Craig
bb43bfb024 Add npm (v3 or greater) as a README dependency 
Experienced an error to do with the way npm handles its dependency
paths.
[Basic idea is here](https://docs.npmjs.com/how-npm-works/npm3#npm-v3-dependency-resolution).

My npm v2.x.x was failing because it couldn't find my Hogan library.
Upgrading npm, nuking `node_modules` and reinstalling/rebuilding
finally fixed everything.
 2016-11-21 17:24:26 +00:00
Chris Hill-Scott
9cf1fea395 Merge pull request #1032 from alphagov/feedback-page 
Added route to the system status page to the feedback page
 2016-11-21 15:02:16 +01:00
Pete Herlihy
5fa7bb78ef Added route to the system status page to the feedback page 2016-11-21 11:25:57 +00:00
Chris Hill-Scott
01852d84e4 Merge pull request #1031 from alphagov/quis-patch-1-1 
Fix heading spacing on dashboard with 0 templates
 2016-11-18 16:52:09 +01:00
Chris Hill-Scott
b6ad19fa31 Fix heading spacing on dashboard with 0 templates 2016-11-18 15:50:56 +00:00
Rebecca Law
a87129e117 Merge pull request #1030 from alphagov/remove-unused-get_all_notificaitons 
Removed unused method
 2016-11-18 14:55:25 +00:00
Rebecca Law
cebf8e1d65 Removed get_all_notifications method from notification_api_client as it is not longer being used. 2016-11-18 14:39:23 +00:00
Chris Hill-Scott
f30495ce92 Merge pull request #1029 from alphagov/header-pattern 
Use cross-GaaP pattern for admin headers
 2016-11-18 14:47:49 +01:00
Chris Hill-Scott
c4bdd9c4a4 Upgrade GOV.UK template 
GOV.UK crown logo image has been tweaked slightly, so our app didn’t
match newer prototypes.

Changes from here: https://github.com/alphagov/govuk_template/pull/237
 2016-11-18 12:00:02 +00:00
Chris Hill-Scott
257fa7f9bb Shift propositional navigation down 
This seems to be a custom thing done on Tim and Stephen’s prototype to
re-align the proposition nav. Not sure if it’s better or worse.
 2016-11-18 12:00:01 +00:00
Chris Hill-Scott
ee78d0bf59 Move switch service out of black header 
Also following the pattern work that Tim and Stephen have done.
 2016-11-18 12:00:01 +00:00
Chris Hill-Scott
bfe234b535 Use two-weight GOV.UK Notify logo 
Based on the work Tim, Stephen et. al have done on patterns for admin
interfaces across GaaP.

See their prototype here:
https://govuk-product-page-prototypes.herokuapp.com/govuk-notify
 2016-11-18 11:59:40 +00:00
Leo Hemsted
805639a2e1 add test for choose services and related redirects to it 
it'll just show the add service button if you only have archived services
 2016-11-16 13:18:49 +00:00
Leo Hemsted
08881e5bd1 add get_active_services method 
* all current invocations of get_services now call get_active_services
  EXCEPT for platform admin page (where we want to see inactive services
* cleaned up parameter names and unpacking (since *params is unhelpful)
* fixed incorrect kwarg name in conftest
 2016-11-16 11:08:20 +00:00
Leo Hemsted
b885ce9cf4 clean up some usage of active in tests and remove it from service_api_client.create_service (created services are always active) 2016-11-16 11:08:20 +00:00
Leo Hemsted
045864f052 Merge pull request #1022 from alphagov/deactivate-services-plat-admin 
Deactivate services plat admin
 2016-11-16 11:07:52 +00:00