darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-09-08 22:53:12 -04:00
Code Issues Packages Projects Releases Wiki Activity

notify-api-340 remove daily limit

Browse Source
This commit is contained in:
Kenneth Kehl
2023-07-11 12:37:27 -07:00
parent 5fe81bc040
commit c165589d4c
5 changed files with 6 additions and 22 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
app/__init__.py
View File

@@ -311,16 +311,6 @@ def init_app(application):
def _nav_selected(): def _nav_selected():
return navigation return navigation
@application.context_processor
def _attach_current_daily_remaining_messages_per_service():
remaining_messages = 0
if hasattr(current_service, 'message_limit'):
remaining_messages = current_service.message_limit - service_api_client.get_notification_count(
service_id=current_service.id)
return {'daily_remaining_messages': remaining_messages}
@application.context_processor @application.context_processor
def _attach_current_global_daily_messages(): def _attach_current_global_daily_messages():
remaining_global_messages = 0 remaining_global_messages = 0

3
app/config.py
View File

@@ -2,7 +2,6 @@ import json
from os import getenv from os import getenv
import newrelic.agent import newrelic.agent
from notifications_utils import DAILY_MESSAGE_LIMIT
from app.cloudfoundry_config import cloud_config from app.cloudfoundry_config import cloud_config
@@ -45,7 +44,7 @@ class Config(object):
DEFAULT_SERVICE_LIMIT = 50 DEFAULT_SERVICE_LIMIT = 50
GLOBAL_SERVICE_MESSAGE_LIMIT = DAILY_MESSAGE_LIMIT GLOBAL_SERVICE_MESSAGE_LIMIT = 250000
EMAIL_EXPIRY_SECONDS = 3600 # 1 hour EMAIL_EXPIRY_SECONDS = 3600 # 1 hour
INVITATION_EXPIRY_SECONDS = 3600 * 24 * 2 # 2 days - also set on api INVITATION_EXPIRY_SECONDS = 3600 * 24 * 2 # 2 days - also set on api

8
app/notify_client/service_api_client.py
View File

@@ -1,9 +1,6 @@
from datetime import datetime from datetime import datetime
from notifications_utils.clients.redis import ( from notifications_utils.clients.redis import daily_total_cache_key
daily_limit_cache_key,
daily_total_cache_key,
)
from app.extensions import redis_client from app.extensions import redis_client
from app.notify_client import NotifyAdminAPIClient, _attach_current_user, cache from app.notify_client import NotifyAdminAPIClient, _attach_current_user, cache
@@ -528,8 +525,7 @@ class ServiceAPIClient(NotifyAdminAPIClient):
return self.get("/service/{}/data-retention".format(service_id)) return self.get("/service/{}/data-retention".format(service_id))
def get_notification_count(self, service_id): def get_notification_count(self, service_id):
# if cache is not set, or not enabled, return 0 count = 0
count = redis_client.get(daily_limit_cache_key(service_id)) or 0
return int(count) return int(count)

2
app/templates/main_nav.html
View File

@@ -28,8 +28,6 @@
</ul> </ul>
</nav> </nav>
<div> <div>
<p class="usa-body bold margin-bottom-1">Messages Left / Daily Limit</p>
<p>{{ daily_remaining_messages }} / {{ current_service.message_limit }}</p>
<p class="usa--body bold margin-bottom-1">Messages Left Across Services</p> <p class="usa--body bold margin-bottom-1">Messages Left Across Services</p>
<p>{{ daily_global_messages_remaining }}</p> <p>{{ daily_global_messages_remaining }}</p>
</div> </div>

5
tests/app/main/views/test_send.py
View File

@@ -1751,7 +1751,7 @@ def test_upload_csvfile_with_valid_phone_shows_all_numbers(
assert '202 867 0750' not in page.text assert '202 867 0750' not in page.text
assert 'Only showing the first 50 rows' in page.text assert 'Only showing the first 50 rows' in page.text
mock_get_notification_count.assert_called_with(service_id=service_one['id']) mock_get_notification_count.assert_called_with(service_one['id'])
@pytest.mark.parametrize('international_sms_permission, should_allow_international', [ @pytest.mark.parametrize('international_sms_permission, should_allow_international', [
@@ -2084,7 +2084,8 @@ def test_check_messages_back_link(
@pytest.mark.parametrize('num_requested,expected_msg', [ @pytest.mark.parametrize('num_requested,expected_msg', [
(None, '‘example.csv’ contains 1,234 phone numbers.'), (None, '‘example.csv’ contains 1,234 phone numbers.'),
("0", '‘example.csv’ contains 1,234 phone numbers.'), ("0", '‘example.csv’ contains 1,234 phone numbers.'),
("1", 'You can still send 49 messages today, but ‘example.csv’ contains 1,234 phone numbers.') # This used to trigger the too many messages errors but we removed the daily limit
("1", '‘example.csv’ contains 1,234 phone numbers.')
], ids=['none_sent', 'none_sent', 'some_sent']) ], ids=['none_sent', 'none_sent', 'some_sent'])
def test_check_messages_shows_too_many_messages_errors( def test_check_messages_shows_too_many_messages_errors(
mocker, mocker,