Alexey Bezhan
17c239bf3d
Remove accidentally committed app/__ file
2017-12-19 10:23:24 +00:00
kentsanggds
18c4e7df43
Merge pull request #1495 from alphagov/ken-test-manifest-delivery-base
...
Add test for manifest-delivery-base.yml
2017-12-18 16:58:07 +00:00
kentsanggds
b0d0b694e4
Merge branch 'master' into ken-test-manifest-delivery-base
2017-12-18 16:51:00 +00:00
Alexey Bezhan
8e2f2818d7
Merge pull request #1504 from alphagov/change-reply-to-address-for-invites
...
Change reply to address for invitation emails to be invitation sender
2017-12-18 16:11:35 +00:00
kentsanggds
1048bd026a
Merge branch 'master' into ken-test-manifest-delivery-base
2017-12-18 15:21:38 +00:00
Katie Smith
1b82afb6bb
Change reply to address for invitation emails to be invitation sender
...
If someone receives an invitation email for Notify the reply-to address
of the email was the GOV.UK Notify email address. This has been changed
to be the email address of the user who sent the invite.
Pivotal story: https://www.pivotaltracker.com/story/show/153094646
2017-12-18 11:39:21 +00:00
Venus Bailey
3015d4fc07
Merge pull request #1492 from alphagov/add-mmg-inbound-sms-auth
...
Add MMG_INBOUND_SMS_AUTH config variable and auth check
2017-12-15 15:01:10 +00:00
venusbb
a374f94dbb
Merge branch 'add-mmg-inbound-sms-auth' of https://github.com/alphagov/notifications-api into add-mmg-inbound-sms-auth
2017-12-15 12:22:30 +00:00
venusbb
a57264d5c8
Merge branch 'master' of https://github.com/alphagov/notifications-api into add-mmg-inbound-sms-auth
2017-12-15 12:21:24 +00:00
venusbb
ab66f5c0ac
Change MMG username to look at env variable
2017-12-15 12:19:58 +00:00
Leo Hemsted
cc574474eb
Merge pull request #1485 from alphagov/letters-tomorrow
...
put pdfs in tomorrow's dvla bucket after 17:30
2017-12-15 11:55:48 +00:00
Leo Hemsted
1ca252dcf9
put pdfs in tomorrow's dvla bucket after 17:30
...
So if someone sends a letter in the evening, it gets picked up the
next day
2017-12-15 11:47:52 +00:00
Venus Bailey
189692773d
Merge branch 'master' into add-mmg-inbound-sms-auth
2017-12-15 09:21:39 +00:00
venusbb
2e6c46c163
Merge branch 'master' of https://github.com/alphagov/notifications-api into add-mmg-inbound-sms-auth
2017-12-15 09:15:49 +00:00
kentsanggds
a25024f52c
Merge branch 'master' into ken-test-manifest-delivery-base
2017-12-14 17:11:50 +00:00
Ken Tsang
5e1eac1f6f
Add test for manifest-delivery-base.yml
...
- This should ensure that queue names defined in config.py / QueueNames are in the manifest-delivery-base.yml
2017-12-14 17:05:36 +00:00
kentsanggds
42318cc335
Merge pull request #1488 from alphagov/ken-update-letters_pdf-billable_units
...
Update billable units for letters pdf task
2017-12-14 16:30:20 +00:00
kentsanggds
5dc238671f
Merge branch 'master' into ken-update-letters_pdf-billable_units
2017-12-14 16:16:00 +00:00
kentsanggds
cf8770d0c1
Merge pull request #1494 from alphagov/ken-rename-create-letters-pdf-queue
...
Create letters pdf queue was renamed with tasks, but was lost in another merge
2017-12-14 16:14:04 +00:00
Ken Tsang
0ad43f0c5b
Create letters pdf queue was renamed with tasks, but was lost in another merge
...
- needs to be correct name otherwise the delivery worker will not pick up the queue
2017-12-14 16:00:51 +00:00
Leo Hemsted
87ea027824
Merge pull request #1493 from alphagov/error-on-app-crash
...
fix makefile syntax
2017-12-14 15:32:02 +00:00
Leo Hemsted
7d1c4ea722
fix makefile syntax
2017-12-14 15:16:43 +00:00
Ken Tsang
0045cd6b72
Use dao_update_notification to update billable_units for letter notifications
2017-12-14 15:09:25 +00:00
Ken Tsang
f23074596b
Update billable units for letters pdf task
2017-12-14 15:09:25 +00:00
Leo Hemsted
5430eb9fc7
Merge pull request #1491 from alphagov/error-on-app-crash
...
if apps crash on startup, then fail deploy process
2017-12-14 15:06:12 +00:00
Leo Hemsted
c6e6fad01f
if apps crash on startup, then fail deploy process
...
we saw an issue where the app started, then immediately crashed due to
a setup error. However, jenkins had already returned positively, and
the deploy continued.
cf-deploy should fail if the app doesn't start up.
We do this by looking through the cloudfoundry events, and aborting
if there are any `app.crash` events for the new GUID.
2017-12-14 14:23:32 +00:00
Leo Hemsted
304d500078
Merge pull request #1490 from alphagov/leohemsted-patch-1
...
Can't have spaces between queue names in manifest
2017-12-14 13:51:00 +00:00
Katie Smith
a8d3b0952f
Add MMG_INBOUND_SMS_AUTH config variable and auth check
...
Checks authentication header value on inbound SMS requests from
MMG against a list of allowed API keys set in the application
config.
At the moment, we're only logging the attempts without aborting the
requests. Once this is rolled out to production and we've checked
the logs we'll switch on the aborts and add the tests for 401 and 403
responses.
This work has already been done for Firetext in a previous PR:
https://github.com/alphagov/notifications-api/pull/1409
2017-12-14 13:37:50 +00:00
Leo Hemsted
5d3a3ab042
Can't have spaces between queue names
2017-12-14 13:26:38 +00:00
Venus Bailey
a9a8ef8ff1
Merge pull request #1489 from alphagov/vb-callback-seperate-queue
...
Create and use separate queue for callbacks
2017-12-14 11:04:26 +00:00
venusbb
2e7b3cf5e5
Merge branch 'master' of https://github.com/alphagov/notifications-api into vb-callback-seperate-queue
2017-12-14 10:29:31 +00:00
kentsanggds
b37cd69650
Merge pull request #1487 from alphagov/ken-update-delivery-base-letters-pdf-task
...
Add create-letters-pdf-tasks to notify-delivery-worker
2017-12-14 10:27:26 +00:00
venusbb
c81c89a9e6
Merge branch 'master' of https://github.com/alphagov/notifications-api into vb-callback-seperate-queue
2017-12-13 11:56:03 +00:00
venusbb
cef8d6e294
use callback queue
2017-12-13 11:55:08 +00:00
Ken Tsang
fdf97fd8b5
Add create-letters-pdf-tasks to notify-delivery-worker
2017-12-13 11:34:05 +00:00
venusbb
11152ab117
Add new queue for callbacks
2017-12-13 10:57:08 +00:00
Chris Hill-Scott
c143b480d2
Merge pull request #1481 from alphagov/hr
...
Add undocumented support for <hr>s in emails
2017-12-13 10:40:46 +00:00
Chris Hill-Scott
c09511bc41
Merge branch 'master' into hr
2017-12-13 10:28:17 +00:00
Venus Bailey
dbd4f96459
Merge pull request #1458 from alphagov/remove-fragments-from-db
...
remove fragments from db
2017-12-13 10:11:52 +00:00
Pete Herlihy
5b2d62fe10
Merge branch 'master' into hr
2017-12-13 10:06:06 +00:00
venusbb
689ece75a0
db version change
2017-12-13 10:03:40 +00:00
Leo Hemsted
df725aaff0
Merge branch 'master' into remove-fragments-from-db
2017-12-12 17:07:04 +00:00
kentsanggds
d18b81093c
Merge pull request #1484 from alphagov/ken-add-template-preview-to-cf
...
Add template preview host url and key to cf config
2017-12-12 17:01:15 +00:00
Pete Herlihy
c8f7f3d0fe
Merge branch 'master' into hr
2017-12-12 16:38:30 +00:00
Leo Hemsted
ea71af695d
Merge branch 'master' into remove-fragments-from-db
2017-12-12 16:32:07 +00:00
kentsanggds
8081ba9438
Merge branch 'master' into ken-add-template-preview-to-cf
2017-12-12 14:57:04 +00:00
Ken Tsang
ebfd78f3cf
Add template preview host url and key to cf config
2017-12-12 14:53:38 +00:00
kentsanggds
ea8135c69a
Merge pull request #1479 from alphagov/ken-add-letters-pdf-task
...
Add letters pdf task & api
2017-12-12 13:37:25 +00:00
Ken Tsang
da93ba296e
Refactored notifications_dao
...
- Introduce a `_raise` flag for `get_notification_by_id` so that sql alchemy will raise the NoResults error rather than the app
- Refactor `dao_set_created_live_letter_api_notifications_to_pending` to use a join for getting services that don't have `letters_as_pdf` as marginally faster.
2017-12-12 12:07:55 +00:00
Ken Tsang
ab72d6f555
Call the create letters pdf task in api calls for services with letters as pdf
2017-12-12 12:07:55 +00:00
