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

Remove redundant sorting when editing SMS ratios

Browse Source 
This is irrelevant since we no longer pick a "first" provider.
This commit is contained in:
Ben Thorner
2022-04-06 17:07:02 +01:00
parent dd0022968c
commit 264bf8302f
1 changed files with 3 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
app/main/views/providers.py
View File

@@ -1,5 +1,4 @@
from datetime import datetime from datetime import datetime
from operator import itemgetter
from flask import render_template, url_for from flask import render_template, url_for
from werkzeug.utils import redirect from werkzeug.utils import redirect
@@ -46,13 +45,11 @@ def add_monthly_traffic(domestic_sms_providers):
@main.route("/provider/edit-sms-provider-ratio", methods=['GET', 'POST']) @main.route("/provider/edit-sms-provider-ratio", methods=['GET', 'POST'])
@user_is_platform_admin @user_is_platform_admin
def edit_sms_provider_ratio(): def edit_sms_provider_ratio():
providers = [
providers = sorted([
provider provider
for provider in provider_client.get_all_providers()['provider_details'] for provider in provider_client.get_all_providers()['provider_details']
if provider['notification_type'] == 'sms' if provider['notification_type'] == 'sms' and provider['active']
and provider['active'] ]
], key=itemgetter('identifier'))
form = AdminProviderRatioForm(providers) form = AdminProviderRatioForm(providers)