darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-09-10 18:23:01 -04:00
Code Issues Packages Projects Releases Wiki Activity
1,477 Commits 122 Branches 63 Tags
4f768b53e6964b3907c9f270b6426269496c1919
Commit Graph
1477 Commits
This Branch
This Branch
All Branches
Author SHA1 Message Date
minglis 4f768b53e6 Merge pull request #658 from alphagov/drop-status-column 
Drop the status column
 2016-09-13 14:46:38 +01:00
minglis d2b936e631 Merge pull request #667 from alphagov/delete-csv-file-after-7-days 
Delete csv file after 7 days
 2016-09-13 09:03:48 +01:00
Rebecca Law 94e78c9c33 Merge pull request #669 from alphagov/accept-invite-tests 
Fix a bug in the accept invitation that was throwing a 500.
 2016-09-12 16:52:32 +01:00
Rebecca Law f1b9702dfb Fix codestyle 2016-09-12 16:41:48 +01:00
Martyn Inglis e0d46d821d Merge branch 'master' into delete-csv-file-after-7-days 
Conflicts:
	app/notifications/rest.py
 2016-09-12 16:33:01 +01:00
Rebecca Law bab9360519 Fix a bug in the accept invitation that was throwing a 500. 
- If the token for invitation was expired, the method would throw a 500 while creating the erorr message.
- Added unit tests.
 2016-09-12 16:16:02 +01:00
minglis af70d6d15c Merge pull request #666 from alphagov/write-to-postgres-not-sqs 
Write to postgres not sqs
 2016-09-09 11:29:16 +01:00
Chris Hill-Scott 3ed97f15a1 Merge pull request #668 from alphagov/update-api-client 
Update Python client
 2016-09-09 09:33:25 +01:00
Martyn Inglis 5f72034577 Renamed and fixed the migration script to be in sequence 2016-09-08 16:12:15 +01:00
Martyn Inglis e099edae51 Merge branch 'master' into drop-status-column 2016-09-08 16:11:18 +01:00
Chris Hill-Scott 1e235fc0cf Update Python client 
Just so that nobody else has to do it.

Implements:
- [x] https://github.com/alphagov/notifications-python-client/pull/29

Which is a breaking change requiring the renaming of method arguments.
 2016-09-08 16:08:30 +01:00
Martyn Inglis a6bfc48478 Merge branch 'master' into write-to-postgres-not-sqs 2016-09-08 16:00:31 +01:00
Martyn Inglis dde22bc58b Implemented deleted notification if SQS write fails 2016-09-08 16:00:18 +01:00
Martyn Inglis a3f3f3e279 Changed the FROM_NUMBER on preview and stage to be shorter 2016-09-08 10:44:14 +01:00
minglis c50a0200d4 Merge pull request #664 from alphagov/rationalise-config 
Simplified the config. Aim is to get the actual secrets in credstash …
 2016-09-08 10:27:31 +01:00
Martyn Inglis 9ee5c661b0 Changed as per @chs comment. Using cls based not explicity calling type 2016-09-08 09:21:32 +01:00
Martyn Inglis b4d40dcd18 Merge branch 'master' into write-to-postgres-not-sqs 
Conflicts:
	app/notifications/rest.py
	tests/app/notifications/rest/test_send_notification.py
 2016-09-08 09:15:07 +01:00
Martyn Inglis bdc1d464bf Fixed up a few things as per pull request review: 
- Moved email domain and css bucket into dev config, not default

- deleted duplicated property

- removed unused property
 2016-09-08 09:07:43 +01:00
imdadahad b1013d8d31 Merge pull request #660 from alphagov/update-readme 
Update README to include additonal instructions
 2016-09-07 16:05:24 +01:00
Martyn Inglis 852e207478 Adds new scheduled task to delete CSV files. 
Deletes files for jobs older than 7 days, matching delete process for the actual notifications

Runs 1 minutes past the hour at midnight, 1 and 2 am.
 2016-09-07 15:36:59 +01:00
Martyn Inglis c3657839e4 New jobs dao method to get jobs that are older than a certain number of days. 
Used in deleting CSV files scheduled task
 2016-09-07 15:36:07 +01:00
Martyn Inglis f506f6751e Removed unused import 2016-09-07 15:35:25 +01:00
Martyn Inglis 0ecd6f2737 Removed the 'delete job' tasks from the tasks.py file 
- this was called after processing a job.
- now to be called  on a schedule.
 2016-09-07 15:35:12 +01:00
imdadahad 6cbd5396ea Merge pull request #662 from alphagov/remove-send-notification-tests 
Remove rate limiting tests
 2016-09-07 15:18:44 +01:00
Martyn Inglis cf107d649f Merge branch 'master' into write-to-postgres-not-sqs 
Conflicts:
	app/celery/tasks.py
 2016-09-07 13:51:21 +01:00
Martyn Inglis 770b0c2838 Update rest end point to wrote to the DB not the queue. 
We wrote to the queue as a performance optimisation , however dev found it confusing to not have immediate access to the notification as it may be perished some minutes later under periods of load. Additionally we had a couple of DB issues which led to us dropping notifications.

Pushing the DB write to earlier in the flow makes the system a little  more robust in the early days, we may want to change this when the traffic increases.
 2016-09-07 13:47:22 +01:00
Martyn Inglis 255ffbd7f6 Tasks now use the new builder method to construct the DB object. 2016-09-07 13:45:37 +01:00
Martyn Inglis 99250b2248 Adds a builder method to the notification object. 
This is used to construct a notification from the sorts of data an API call provides. This is used in both the db-email / db-sms tasks and the notifications rest endpoint to construct the notification DB object.
 2016-09-07 13:44:56 +01:00
imdadahad 64645d374a Merge pull request #665 from alphagov/fix-test-key-external-number-issue 
Fix test key external number issue
 2016-09-07 11:56:22 +01:00
Imdad Ahad 41681ac001 Fix mocker syntax issue 2016-09-07 11:17:27 +01:00
Imdad Ahad 94708bdee8 Fix indentation issue 2016-09-07 11:03:16 +01:00
Imdad Ahad 840b99b277 Fix issue where test key cannot send messages to external numbers 2016-09-07 09:57:20 +01:00
Martyn Inglis d8bfad1ce7 Added prefix to test env file for completeness 2016-09-07 09:37:45 +01:00
Martyn Inglis c1b00a5f0c Simplified the config. Aim is to get the actual secrets in credstash to be env specific, and not the random collection of things we have at the moment. 
Secret definition also includes env specific things such as URLs / Queue prefixes / URLs for providers and so on.
 2016-09-07 09:35:31 +01:00
Martyn Inglis 259fb0cb62 Slightly changed to only unlimited the live services. 
Logic:
- live services don't check days limit for now
- restricted services check limits
(caveat) simulate keys aren't checking day limit even in restricted mode.
 2016-09-06 14:21:36 +01:00
Rebecca Law 727c00af21 Merge pull request #659 from alphagov/update-requirements 
Update requirements
 2016-09-06 14:16:31 +01:00
Rebecca Law 62cd234510 Merge pull request #663 from alphagov/fix-cancelled-jobs-migrations 
Fix mis-matched database migrations
 2016-09-06 13:18:27 +01:00
Chris Hill-Scott 1bf37888dd Fix mis-matched database migrations 
The cancelled job migration originally went in as version `0051`.

But `0051_set_job_status` went in first in
e3fa8bffd7

This meant that Alembic couldn’t work out what order to run the
migrations in, and blew up.

This commit puts the scheduled job migration after all others.
 2016-09-06 13:07:20 +01:00
Chris Hill-Scott 9318eec3a5 Merge pull request #661 from alphagov/update-requirements-1 
Update requirements
 2016-09-06 12:12:52 +01:00
Chris Hill-Scott be9fd96d03 Merge pull request #656 from alphagov/add-cancel-job-endpoint 
Add an endpoint to cancel a job
 2016-09-06 12:11:32 +01:00
Imdad Ahad d6a8780515 Remove send notification test 2016-09-06 11:39:22 +01:00
Chris Hill-Scott 3b6905b98d Update requirements 
Brings in:
- [x] https://github.com/alphagov/notifications-utils/pull/67
 2016-09-06 10:53:34 +01:00
Imdad Ahad d8a8a69aff Update README to include additonal instructions 2016-09-06 10:30:57 +01:00
Rebecca Law 96a1d1a135 Update celery requirement 2016-09-05 17:00:16 +01:00
Rebecca Law 4d1c34fcce A little bit of clean up. 
- Remove the deploy to staging | live links in the README
- Update most of the outdated requirements. Left Flask update out for now.
 2016-09-05 16:45:54 +01:00
Martyn Inglis cf3ca764c5 Drop the status column 2016-09-05 16:16:32 +01:00
minglis 1d5db00b87 Merge pull request #657 from alphagov/drop-model-before-column 
Changed 0052 to not drop the column, rather set it to allow nulls.
 2016-09-05 14:46:46 +01:00
Martyn Inglis 22844d1bf2 Changed 0052 to not drop the column, rather set it to allow nulls. 
This allows existing code to run.
 2016-09-05 13:22:45 +01:00
minglis 64bb12c866 Merge pull request #640 from alphagov/scheduled-delivery-of-jobs-cleanup 
Scheduled delivery of jobs cleanup
 2016-09-05 12:29:10 +01:00
Leo Hemsted 6ceb80d32f Merge pull request #651 from alphagov/contract-tests 
Contract tests
 2016-09-05 11:31:29 +01:00