Imdad Ahad
ea2f838510
Fix typo
2017-08-31 11:10:54 +01:00
Katie Smith
49a6bfc06b
Send '0', not 'null', to perf platform if no notifications are sent
2017-08-30 16:02:30 +01:00
Katie Smith
a1a5fdedb1
Send results of processing-time query to performance platform
2017-08-30 14:40:10 +01:00
Katie Smith
132d65bc75
Add query to get processing time stats for performance platform
...
We are only interested in API notifications, not including test
messages. Letters are not included.
2017-08-30 14:40:10 +01:00
Chris Hill-Scott
4f0443041d
Merge pull request #1213 from alphagov/remove-spec
...
Remove Swagger specification
2017-08-29 14:19:31 +01:00
Leo Hemsted
cf2d803db1
Merge pull request #1205 from alphagov/perf-stats
...
Perf stats refactor
2017-08-29 10:44:09 +01:00
Chris Hill-Scott
d0be79cc4c
Remove Swagger specification
...
This was only ever a spike into what it might look like to document
Notify’s API with Swagger (see
7c3d25a87a ).
It’s no longer updated, and only talks about version 1 of the public
API.
Keeping it around now is just a liability, and gives us additional Pyup
upgrades to deal with.
2017-08-29 10:32:37 +01:00
Leo Hemsted
c863b33c8d
make sure letters are handled properly by the v1 api
2017-08-25 16:47:17 +01:00
kentsanggds
549601a6df
Merge pull request #1200 from alphagov/ken-refactor-inbound_numbers
...
Refactor inbound numbers
2017-08-25 13:27:16 +01:00
Leo Hemsted
e85b621cbc
make perf platform client handle more stuff sensibly
...
specifically, all of the performance platform specific data layout now
happens in performance_platform_client.py - stuff like setting the
_timestamp, period etc, and the perf platform-specific nomenclature is
all handled there.
2017-08-24 17:10:42 +01:00
Rebecca Law
f9ee8bba91
Merge pull request #1209 from alphagov/remove-unused-endpoint
...
Removed unused endpoint and dao methods.
2017-08-24 16:32:37 +01:00
Rebecca Law
717c0510a3
Merge pull request #1202 from alphagov/v1-send-letter-error-msg
...
Added a proper error response if the notification type is not supported.
2017-08-24 16:31:21 +01:00
Rebecca Law
17f62723fa
Removed unused endpoint and dao methods.
2017-08-24 16:09:48 +01:00
Ken Tsang
1b35731fb2
Refactor code
...
- updated dao_get_letter_jobs_by_status to dao_get_letter_job_ids_by_status to return array of strings
2017-08-24 11:57:46 +01:00
Leo Hemsted
412c87cfc8
pycodestyle
2017-08-24 10:52:47 +01:00
Leo Hemsted
bd2682b521
add new performance-platform section to cf config
...
it's a new cf-service we've got to create, that contains endpoints
and the bearer tokens for them.
2017-08-23 18:14:35 +01:00
Ken Tsang
451c5a9f1a
Made celery task arg a tuple
2017-08-23 18:05:48 +01:00
Leo Hemsted
89f4f5173e
refactor performance platform code
...
so that it doesn't appear generic when it's actually specific to
sending the daily notification totals. To do this, split it out into a
separate performance_platform directory, containing the business logic,
and make the performance_platform_client incredibly thin - all it
handles is adding ids to payloads, and sending stats.
Also, some changes to the config (not all done yet) since there is one
token per endpoint, not one for the whole platform as we'd previously
coded
2017-08-23 17:37:29 +01:00
Rebecca Law
d4422dd35f
Added a proper error response if the notification type is not supported.
2017-08-23 14:56:03 +01:00
Ken Tsang
d99ab329eb
Refactored code to use inbound_number.number
...
- Removed filter on sms_sender for `dao_fetch_services_by_inbound_number`
2017-08-23 14:14:32 +01:00
Ken Tsang
c3ce828635
Updated letter jobs schedule to 5:30pm
2017-08-23 13:22:07 +01:00
Ken Tsang
4fb5e68ce7
Refactor code
...
- Created TaskNames for DVLA_FILES rather than have DVLA_FILES in QueueNames
- Removed PROCESS_FTP from all_queues() as this was causing problems in picking up letter job tasks
- Created test to ensure that we don't arbitrarily add queue names to all_queues
2017-08-23 13:22:07 +01:00
Ken Tsang
41fce57932
Removed PROCESS_FTP from all_queues
2017-08-23 13:22:07 +01:00
Ken Tsang
b2b74e2b5d
Fixed bug in letter job schedule
2017-08-23 13:22:07 +01:00
Ken Tsang
62172ce39b
Updated schedule to run at 5:15pm
2017-08-23 13:22:07 +01:00
Ken Tsang
18881cd580
Add scheduled letter jobs
2017-08-23 13:22:07 +01:00
Ken Tsang
8c4a084a0d
Refactor letters filename
2017-08-23 13:22:07 +01:00
Rebecca Law
789492060c
Merge pull request #1198 from alphagov/fix-redis-error
...
Stop redis cache from throwing an error
2017-08-23 09:38:20 +01:00
Leo Hemsted
12cf36f6fa
Merge pull request #1142 from alphagov/log-ip-block
...
log ip address as separate field to allow analysis of messages
2017-08-22 17:01:54 +01:00
Leo Hemsted
ae683cad6e
Merge pull request #1192 from alphagov/deps
...
update dependencies
2017-08-22 15:39:49 +01:00
Rebecca Law
bf6e671e96
The logs kept reporting a data error from the redis client. This would happen any time we set the cache with empty cache values.
...
Added a check for values before setting the cache
2017-08-22 14:59:50 +01:00
Chris Hill-Scott
66d1c3ca03
Merge pull request #1194 from alphagov/no-test-key-search
...
Exclude test keys when searching by recipient
2017-08-22 12:31:00 +01:00
kentsanggds
6cd6e70fe4
Merge pull request #1193 from alphagov/ken-update-inbound_numbers-service_id
...
Update inbound_numbers to services that have sms_sender in inbound_numbers
2017-08-22 10:02:57 +01:00
Chris Hill-Scott
bdc935a8b1
Exclude test keys when searching by recipient
...
The activity page doesn’t show notifications sent with a test key.
However it _does_ when you search by recipient. This is confusing
and inconsistent.
2017-08-21 17:38:21 +01:00
Ken Tsang
27ccbb0cd1
Removed trailing whitespace
2017-08-21 13:31:35 +01:00
Leo Hemsted
c2152f9cb8
remove billable-units endpoint
...
it wasn't used anywhere, and the return value changed when flask was upgraded
2017-08-21 10:57:15 +01:00
Leo Hemsted
c36e50bef1
update dependencies
2017-08-18 17:02:31 +01:00
Imdad Ahad
e05160bdfb
Aggregate monthly totals from billing data
2017-08-18 16:12:01 +01:00
Imdad Ahad
58baa9919c
Merge pull request #1190 from alphagov/imdad-feat-get-monthly-usage-from-billing-table
...
Get monthly usage breakdown from the correct table
2017-08-18 14:21:12 +01:00
Imdad Ahad
d7d2ff7af4
Query the correct table for monthly billing usage:
...
* Query `MonthlyBilling` for the monthly usage and transform
response for admin
2017-08-18 13:38:53 +01:00
Imdad Ahad
2371c2c9a7
Add helper to get months for a financial year:
...
* No tests have been added as this was taken from the admin
where it is already tested thoroughly.
* TODO: Move to utils
2017-08-18 12:45:05 +01:00
Ken Tsang
39cb66a882
Update script to only update unassigned inbound
2017-08-18 11:49:05 +01:00
Ken Tsang
e6582b90be
Add db command to update inbound_numbers
...
Updates the service_id with the service.id that matches the sms_sender
2017-08-18 11:49:05 +01:00
kentsanggds
3bfbb9d71f
Merge pull request #1178 from alphagov/ken-use-inbound-number
...
Use inbound number for delivering sms and to show on service settings
2017-08-18 11:42:52 +01:00
Ken Tsang
5bb20e41b3
Update dao_fetch_services_by_ss_sender
2017-08-17 18:15:22 +01:00
Chris Hill-Scott
953e2ae5bd
Merge pull request #1180 from alphagov/return-template-name
...
Return template name in public API
2017-08-17 12:44:24 +01:00
Imdad Ahad
f40889e6e0
This adds new endpoints to retrieve billing data from the new table:
...
1. Create a separate billing blueprint to house these endpoints
2. Return monthly breakdown in same format as we did before
3. Return yearly breakdown but only return {billing units, rate,
notification_type}. Admin only makes use of these.
2017-08-17 11:10:42 +01:00
Ken Tsang
9ab1dfd6d0
Just order by updated_at for inbound_numbers
2017-08-16 16:28:21 +01:00
Ken Tsang
902b28e00f
Refactor to check active flag
2017-08-16 16:27:42 +01:00
Imdad Ahad
9b88dadb83
Update command populate billing data for services that require it
2017-08-16 15:15:34 +01:00