diff --git a/app/templates/views/providers/providers.html b/app/templates/views/providers/providers.html index 027afc951..b2d113e8e 100644 --- a/app/templates/views/providers/providers.html +++ b/app/templates/views/providers/providers.html @@ -12,6 +12,8 @@ Providers

SMS

+ Change priority + {% call(item, row_number) list_table( domestic_sms_providers, caption="Domestic SMS providers", @@ -21,7 +23,7 @@ Providers field_headings_visible=True ) %} - {{ link_field(item.display_name, url_for('main.view_provider', provider_id=item.id)) }} + {{ text_field(item.display_name) }} {{ text_field(item.priority) }} @@ -37,8 +39,6 @@ Providers {{ optional_text_field(item.created_by_name, default='None') }} - {{ link_field('change', url_for('main.edit_provider', provider_id=item.id)) }} - {% endcall %}

Email

diff --git a/tests/app/main/views/test_providers.py b/tests/app/main/views/test_providers.py index 372ef2d6d..0337ea522 100644 --- a/tests/app/main/views/test_providers.py +++ b/tests/app/main/views/test_providers.py @@ -174,26 +174,24 @@ def test_should_show_all_providers( domestic_sms_first_row = domestic_sms_table.tbody.find_all('tr')[0] table_data = domestic_sms_first_row.find_all('td') - assert table_data[0].find_all("a")[0]['href'] == '/provider/6005e192-4738-4962-beec-ebd982d0b03f' + assert not table_data[0].find_all("a") assert table_data[0].text.strip() == "Domestic SMS Provider" assert table_data[1].text.strip() == "1" assert table_data[2].text.strip() == "42" assert table_data[3].text.strip() == "True" assert table_data[4].text.strip() == "16 January at 3:20pm" assert table_data[5].text.strip() == "Test User" - assert table_data[6].find_all("a")[0]['href'] == '/provider/6005e192-4738-4962-beec-ebd982d0b03f/edit' domestic_sms_second_row = domestic_sms_table.tbody.find_all('tr')[1] table_data = domestic_sms_second_row.find_all('td') - assert table_data[0].find_all("a")[0]['href'] == '/provider/0bd529cd-a0fd-43e5-80ee-b95ef6b0d51f' + assert not table_data[0].find_all("a") assert table_data[0].text.strip() == "Second Domestic SMS Provider" assert table_data[1].text.strip() == "2" assert table_data[2].text.strip() == "58" assert table_data[3].text.strip() == "True" assert table_data[4].text.strip() == "None" assert table_data[5].text.strip() == "None" - assert table_data[6].find_all("a")[0]['href'] == '/provider/0bd529cd-a0fd-43e5-80ee-b95ef6b0d51f/edit' domestic_email_first_row = domestic_email_table.tbody.find_all('tr')[0] domestic_email_table_data = domestic_email_first_row.find_all('td')