darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-01-03 17:23:54 -05:00
Code Issues Packages Projects Releases Wiki Activity
296 Commits 127 Branches 60 Tags
f8ac14ea300d2f6b3216f659900bd6431097c698
Commit Graph

296 Commits

This Branch
This Branch
All Branches
Author SHA1 Message Date
Martyn Inglis
f8ac14ea30 Update model 
- adds email_from field to the service model
- adds subject_line to the template model

These are unique and not null fields, so there is a migration here for email_from to populate it.
 2016-02-19 15:52:19 +00:00
NIcholas Staples
13b7a89c8c Merge pull request #82 from alphagov/db-error-handling 
Wire up error handlers.
 2016-02-17 17:23:27 +00:00
Adam Shimali
4f33b6f406 Wire up error handlers. 
Replace some 400s with more appropriate 500s.

DAO methods that cause unexpected exceptions get caught and
logged by errors.py 500 error handler.
 2016-02-17 17:04:50 +00:00
NIcholas Staples
b98d8b9996 Merge pull request #80 from alphagov/fire-text-client 
Fire text client
 2016-02-17 14:55:02 +00:00
Martyn Inglis
53bdedf8f3 Test properties for firetext 2016-02-17 14:27:28 +00:00
Martyn Inglis
36eee56f17 Adding boto for python2 as Celery seems to need it. 2016-02-17 14:03:40 +00:00
Martyn Inglis
837c9b7cdb Removed logging, and make fire text only client. 2016-02-17 13:59:01 +00:00
Rebecca Law
c7cebc4d00 Merge pull request #78 from alphagov/move-sms-notifications-into-celery 
Move sms notifications into celery
 2016-02-17 13:29:45 +00:00
Martyn Inglis
226459132a Basic (Very basic) implementation of the fire text API. 
[https://www.firetext.co.uk/docs#sendingsms](https://www.firetext.co.uk/docs#sendingsms)

Not to be merged. This API has a limit on it at the moment that will need to be removed before it is used in anger.
 2016-02-17 12:57:51 +00:00
Martyn Inglis
58ae041b64 AWS scripts for celery 2016-02-17 11:36:14 +00:00
Martyn Inglis
e59e66fa21 Run celery needed to read in secrets 2016-02-17 11:25:57 +00:00
Martyn Inglis
cebca61f8e Travis deploy group for delivery version of api 2016-02-17 11:21:49 +00:00
Martyn Inglis
95cdfe08d1 Updated logging in run-celery script to log to application logs 
- poll every second
- visibility of 1 minute
 2016-02-17 11:18:27 +00:00
Rebecca Law
5f4b6a7db9 Fix db.py script for credstash secrets 2016-02-17 11:01:15 +00:00
NIcholas Staples
ab1279116f Merge pull request #77 from alphagov/clarify-readme 
Updated sample values for secret keys in environment.sh to match
 2016-02-17 10:29:44 +00:00
Martyn Inglis
e0e47b40fc Setup celery config 2016-02-17 10:22:25 +00:00
Adam Shimali
914b650a46 Updated sample values for secret keys in environment.sh to match 
that suggested in delivery app.

Called out the reason for unique prefixes or queue names.
 2016-02-17 10:20:40 +00:00
Chris Hill-Scott
bb88fd79dc Merge pull request #76 from alphagov/update-readme 
Update README with set up instructions.
 2016-02-17 09:58:21 +00:00
Rebecca Law
8b4641021a Update README with set up instructions. 2016-02-17 09:49:36 +00:00
Martyn Inglis
85a8e6d2b4 Added moto back for methods not yet celery'd 2016-02-17 09:47:34 +00:00
Martyn Inglis
ec8955503f Twilio params into setup file for tests 2016-02-17 09:34:57 +00:00
Martyn Inglis
73d0e67e68 Removed alpha api client 2016-02-17 09:22:06 +00:00
Martyn Inglis
cb265896a7 Merge branch 'master' into move-sms-notifications-into-celery 
Conflicts:
	app/__init__.py
	tests/conftest.py
 2016-02-17 09:14:37 +00:00
Martyn Inglis
1311af5920 Twilio client takes a 'to' number not a notification 2016-02-17 09:12:13 +00:00
Martyn Inglis
a2341be0e2 Don't send the SMS if we have failed to save in the database 2016-02-16 17:42:04 +00:00
Martyn Inglis
0933e5c647 Building tests for the tasks class 2016-02-16 17:17:02 +00:00
minglis
c1e3317967 Merge pull request #75 from alphagov/update-config 
Update config
 2016-02-16 16:39:41 +00:00
Rebecca Law
af86520e95 Add verify_code_email_address 2016-02-16 16:03:30 +00:00
Rebecca Law
08ba5de61b Update config 
Source the configuration from an environment file, this way it is similar to how the aws environment works
 2016-02-16 16:01:50 +00:00
Martyn Inglis
18d63e241b Extracted serialiser for encryption into a flask module 
- allows mocking easier
- shared across methods
- not built everytime
 2016-02-16 15:28:30 +00:00
Martyn Inglis
e42da7dd54 Fixing up tests to validate the call to the celery tasks. 
- mocker used to test call or otherwise of the task
- no new tests just a spring clean
 2016-02-16 14:06:56 +00:00
Chris Hill-Scott
7ff5f6c45a Merge pull request #74 from alphagov/update_from_email_address 
Update to config to pull out from email address.
 2016-02-16 11:56:56 +00:00
Martyn Inglis
655beddba6 Fixed up the get_notitication endpoint 
- returns a notification
 2016-02-16 11:22:44 +00:00
Nicholas Staples
20650a91e0 Update to config to pull out from email address. 2016-02-16 11:10:02 +00:00
Martyn Inglis
223cb8c2dd Made SMS messages go through celery 
- twilio client pulled in from delivery app
- made method to perform task
 2016-02-15 16:01:14 +00:00
NIcholas Staples
025a6fb47e Merge pull request #72 from alphagov/fix-template-validation 
Fix validation of template for service.
 2016-02-15 15:10:16 +00:00
Rebecca Law
fcaed04918 Fix validation of template for service. 
When the delivery app creates the sms on behalf of the service,
the validation was failing.
 2016-02-15 15:02:19 +00:00
Martyn Inglis
ffbe94f390 Merge branch 'master' into celery-spike 
Conflicts:
	app/notifications/rest.py
 2016-02-15 11:12:07 +00:00
Martyn Inglis
16c5e7bf10 Celery task added 2016-02-15 11:11:20 +00:00
Chris Hill-Scott
e7ef5945e5 Merge pull request #71 from alphagov/validate_template_for_a_service 
Validate template for a service
 2016-02-12 15:14:24 +00:00
Nicholas Staples
1eb18e7f07 Code review fix. 2016-02-12 14:08:48 +00:00
Nicholas Staples
918c561726 Code added to now check service id matches the authorization token service for sending an sms. 2016-02-12 11:13:54 +00:00
Nicholas Staples
4865622128 Merge remote-tracking branch 'origin/master' into validate_template_for_a_service 2016-02-12 10:22:17 +00:00
NIcholas Staples
91164b0afd Merge pull request #70 from alphagov/return-notification-id 
Return notification is when adding notification to queue
 2016-02-12 09:48:56 +00:00
Martyn Inglis
fcc1585fdf Merge branch 'master' into celery-spike 
Conflicts:
	app/__init__.py
	app/notifications/rest.py
	config.py
	wsgi.py
 2016-02-12 09:36:49 +00:00
Rebecca Law
0cd5fa278b Fix a intermittent test. 
Removed the need for sample_admin_service_id in service/test_rest
 2016-02-10 13:09:36 +00:00
Rebecca Law
2db0f9737e Added a test to check endpoint works with an id in payload. 2016-02-10 12:46:52 +00:00
Rebecca Law
a01828a6d0 Return notification_id on create notification endpoints 
- /notification/sms
 - /notification/email
 - /notificaiton/sms/service/<service_id>
Update message attribute on SQS to notification_id from message_id
 2016-02-10 12:46:52 +00:00
minglis
ea22e53453 Merge pull request #68 from alphagov/bump-client 
Updated python client verion
 2016-02-10 12:38:33 +00:00
Martyn Inglis
a1be4e3ca5 Updated python client verion 2016-02-10 12:33:54 +00:00