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 constraint on multiple providers

Browse Source 
We can now handle a single provider (with a priority of 100%).  I
don't think we need to handle the case of no providers.
This commit is contained in:
Ben Thorner
2022-04-06 17:05:05 +01:00
parent c4e1f40b43
commit dd0022968c
1 changed files with 1 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -1,7 +1,7 @@
from datetime import datetime from datetime import datetime
from operator import itemgetter from operator import itemgetter
from flask import abort, render_template, url_for from flask import render_template, url_for
from werkzeug.utils import redirect from werkzeug.utils import redirect
from app import provider_client from app import provider_client
@@ -56,9 +56,6 @@ def edit_sms_provider_ratio():
form = AdminProviderRatioForm(providers) form = AdminProviderRatioForm(providers)
if len(providers) < 2:
abort(400)
if form.validate_on_submit(): if form.validate_on_submit():
for provider in providers: for provider in providers:
field = getattr(form, provider['identifier']) field = getattr(form, provider['identifier'])