darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-09-10 18:22:37 -04:00
Code Issues Packages Projects Releases Wiki Activity
17,660 Commits 60 Branches 61 Tags
rollback-branch
Commit Graph
17660 Commits
This Branch
This Branch
All Branches
Author SHA1 Message Date
Rebecca Law fd11e863da Fix bug with variable names 2016-01-22 11:21:30 +00:00
Nicholas Staples ac433835f0 Fixed viewing templates. 2016-01-22 11:20:39 +00:00
Nicholas Staples bb853ee95a Added fixes, displaying templates still needs to be fixed. 2016-01-22 11:14:56 +00:00
Chris Hill-Scott 6f1f514b2b Add restricted mode warning 
For the hack day, we should only let developers use the platform in restricted
mode. This commit adds a banner telling them this.

Can’t get the app running locally, so fingers crossed it actually looks how
I imagine it’s going to look…
 2016-01-22 11:00:44 +00:00
Rebecca Law 6fde5f99c7 Merge pull request #98 from alphagov/user-api-bug-fix 
Incorrect url passed to BaseApiClient which already has base_url.
 2016-01-22 10:42:21 +00:00
Martyn Inglis 79c21388bb Content changes for switching services 2016-01-22 10:38:29 +00:00
Martyn Inglis 860f8e192c Removed side nav links for things we don't need on the hack day. 2016-01-22 10:31:34 +00:00
Adam Shimali 3e19fa50d2 Incorrect url passed to BaseApiClient which already has base_url. 
Also added getter and setter of failed login count, although that is
yet to be synced with api.
 2016-01-22 10:27:14 +00:00
Martyn Inglis 3b19c9b853 Updated sidebar links 2016-01-22 10:25:17 +00:00
Rebecca Law 64af225d2d fix code style 2016-01-22 10:02:41 +00:00
Rebecca Law 4ea59688b8 Remove base url for the token creation. 
Fix variable name.
 2016-01-22 10:01:14 +00:00
Adam Shimali f461e997cf Merge pull request #96 from alphagov/api-keys-flow 
Add pages for create/view/revoke API keys
 2016-01-21 17:27:08 +00:00
Rebecca Law 61893c5c7e Validation on key name is case insenstive 2016-01-21 16:52:01 +00:00
Rebecca Law 81a5164fa5 Fix test to use mock_api_user 2016-01-21 15:56:08 +00:00
Rebecca Law 46e0e3fd1b Merge branch 'master' into api-keys-flow 
Conflicts:
	tests/app/main/views/test_api_keys.py
 2016-01-21 15:50:00 +00:00
Rebecca Law fe6b406de4 Merge pull request #97 from alphagov/complete-sign-in 
Complete sign in
 2016-01-21 15:45:00 +00:00
Chris Hill-Scott a66dd52ca8 Better words 2016-01-21 15:43:31 +00:00
Rebecca Law e7713a8b7f Added a validator so that the key name is unique per service. 2016-01-21 14:15:36 +00:00
Rebecca Law 2b24909ca6 Fix tests for to use mock user for login 2016-01-21 12:43:11 +00:00
Rebecca Law 8737edfad1 Merge branch 'master' of github.com:alphagov/notifications-admin into api-keys-flow 
Conflicts:
	tests/app/main/views/test_api_keys.py
 2016-01-21 12:31:28 +00:00
Adam Shimali 167c7b0f13 Fixed some tests. Some broken mocked tests commented out until later 2016-01-21 12:31:09 +00:00
Rebecca Law 90fca93308 Implementation of api key pages. 
Revoke page will show the correct key name
Show api keys shows a well formatted expiry date
Fix tests for api key endpoints.
 2016-01-21 12:28:05 +00:00
Adam Shimali ee23df2c73 some tests fixed 2016-01-21 12:12:06 +00:00
Adam Shimali 856b6adb56 First slice full sign in flow 2016-01-21 11:33:53 +00:00
NIcholas Staples 33075178e8 Merge pull request #94 from alphagov/verify-user 
Verify activate and login user with sms and email code
 2016-01-21 09:45:19 +00:00
Rebecca Law 41c775cd68 Created api_key_api_client. 
Implementation of create, revoke and show api keys for service.
These calls work, however we still need to fix the tests.
 2016-01-20 17:32:55 +00:00
Adam Shimali 4e28410794 Temporarily comment out tests that are timing out until mocking sorted 2016-01-20 17:24:17 +00:00
Nicholas Staples 39cc76947f Bumped version of notifications-python-client. 2016-01-20 17:12:57 +00:00
Adam Shimali 6a1cc32fd3 Merge pull request #93 from alphagov/send-sms-page-2 
Put message previews on ‘Send SMS’ page
 2016-01-20 16:36:34 +00:00
Adam Shimali 7af53761d9 fixed some mocks in test 2016-01-20 16:36:06 +00:00
Chris Hill-Scott 9784a9936c Add pages for create/view/revoke API keys 
Copying what they’ve done on GOV.UK Pay, we should let users:
- generate as many keys as they want
- only see the key at time of creation
- give keys a name
- revoke any key at any time (this should be a one way operation)

And based on discussions with @minglis and @servingUpAces, the keys should be
used in conjunction with some kind of service ID, which gets encrypted with the
key. In other words the secret itself never gets sent over the wire.

This commit adds the UI (but not the underlying API integration) for doing the
above.
 2016-01-20 16:22:23 +00:00
Adam Shimali fcedae5fe1 Merge conflicts and test fixes 2016-01-20 15:40:06 +00:00
Adam Shimali ca3d3240a6 Verify activate and login user with sms and email code 2016-01-20 15:34:42 +00:00
Adam Shimali 5924500f3e Merge pull request #95 from alphagov/implement_send_sms_and_email 
Implement send sms and email
 2016-01-20 15:24:29 +00:00
Nicholas Staples 75d7110642 Updated to send email through the client send_email. 2016-01-20 14:45:50 +00:00
Chris Hill-Scott b6390b02a3 Remove old template picker 2016-01-20 13:20:16 +00:00
Chris Hill-Scott 9c8dca4487 Add example of template picker to styleguide 2016-01-20 13:19:28 +00:00
Chris Hill-Scott f36a1bb6c9 Make SMS templates plausible for hack day 
This commit replaces the previous SMS templates.

I’ve written a couple of new ones which are plausible for developers on the
hack day:
- one with placeholders
- one without
 2016-01-20 13:12:21 +00:00
Chris Hill-Scott a20b52b12b Remove ‘create template’ link 
Again, because the users of this page are probably not the same ones responsible
for creating templates, and we shouldn’t be encouraging users to create new
templates for each job (defeats the object of having templates).
 2016-01-20 13:12:20 +00:00
Chris Hill-Scott 482abb97d5 Put message previews on ‘Send SMS’ page 
A previous commit removed these to differentiate between this page and the
manage templates page. It turns out that we do want previews on this page
because:
- the users for the two pages might be different—they might only be allowed to
  do one or the other depending what permissions they have
- it’s useful to see what the placeholders in the message are before uploading
  a CSV, to make sure the CSV has the correct column headings

This commit re-adds the message preview with a simpler UI. Discussed with
@antimega and we both agreed that the speech bubble tails on the messages should
go.
 2016-01-20 13:12:20 +00:00
Nicholas Staples ae781bc58d Merge remote-tracking branch 'origin/master' into implement_send_sms_and_email 2016-01-20 11:46:57 +00:00
Nicholas Staples 2cc9dc6995 Work in progress. 2016-01-20 11:46:39 +00:00
NIcholas Staples 596da7016c Merge pull request #92 from alphagov/requirements-fix 
move to proper version of api client library
 2016-01-20 10:49:19 +00:00
Adam Shimali 3895280d52 move to proper version of api client library 2016-01-20 10:22:53 +00:00
NIcholas Staples 6cbeeb0eb6 Merge pull request #91 from alphagov/user-register-via-api 
User registration now creates user via api.
 2016-01-20 09:50:56 +00:00
Nicholas Staples 9266c14f46 Merge remote-tracking branch 'origin/master' into implement_send_sms_and_email 2016-01-20 09:48:25 +00:00
Nicholas Staples 4a73e9315c Implemented send_sms from client. 2016-01-20 09:46:48 +00:00
Adam Shimali 422b1fb5c2 uncomment accidentally commented out tests 2016-01-20 09:43:38 +00:00
Adam Shimali b5cdf86b40 User registration now creates user via api. 
Verification flow is still to be completed.

Foreign key constraint on verify codes to user table removed.
 2016-01-19 22:49:46 +00:00
Martyn Inglis 35b711f956 No db updates on admin app 2016-01-19 21:54:31 +00:00