darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-07-18 21:44:11 -04:00
Code Issues Packages Projects Releases Wiki Activity

Remove redundant code to populate account type

Browse Source 
This field is now only used on the confirmation page of the settings
form, where we pre-populate it with the hyphenated string it expects.
This commit is contained in:
Ben Thorner
2021-06-07 17:36:55 +01:00
parent 5bfe5f86de
commit 9ada8ad11e
1 changed files with 0 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
app/main/forms.py
View File

@@ -2365,21 +2365,6 @@ class GoLiveNotesForm(StripWhitespaceForm):
class ServiceBroadcastAccountTypeField(GovukRadiosField):
# When receiving Python data, eg when instantiating the form object
# we want to convert it from a tuple of
# (service_mode, broadcast_channel, allowed_broadcast_provider)
# to a value to be used in our form such as "live-severe-ee"
def process_data(self, value):
if not value or isinstance(value, str):
return super().process_data(value)
(live, broadcast_channel, allowed_broadcast_provider) = value
account_type = None
if broadcast_channel:
account_type = "live" if live else "training"
account_type += f"-{broadcast_channel}-{allowed_broadcast_provider}"
self.data = account_type
# After validation we split the value back into its parts of service_mode
# broadcast_channel and provider_restriction to be used by the flask route to send to the
# API