Assessment Only spreadsheet upload (#4269)

* work in progress

* linting

* add spec test

* add extension, ensure Non-UK sorted correctly

* spec fixes

* wip

* more wip

* handle international numbers and improve handling of missing regions

* Reorder regions so national comes before non-uk

* tweak formatting of extension

* Tweak order of National; regenerate list

app/components/grouped_cards/card_component.html.erb

@@ -15,7 +15,7 @@
     <% for field, value in fields %>
       <div class="group__card__fields">
         <span><%= field.humanize %>:</span>
-        <span><%= linked_value(value) %></span>
+        <%= linked_value(field, value) %>
       </div>
     <% end %>
   </div>

app/components/grouped_cards/card_component.rb

@@ -25,11 +25,28 @@ def linked_header
     end
 
     def fields
-      @data.without("header", "link", "status")
+      @data.without("header", "link", "status", "extension", "international_phone")
     end
 
-    def linked_value(value)
-      Rinku.auto_link(h(value)).html_safe
+    def linked_value(field, value)
+      tags = []
+
+      if field == "telephone" && value.present?
+        tags << if @data["international_phone"].present?
+                  # Although we present the national telephone number, we need to use the
+                  # international_phone value in the URL to support overseas visitors
+                  tag.span { link_to(value, "tel:#{@data['international_phone']}") }
+                else
+                  tag.span { link_to(value, "tel:#{value}") }
+                end
+        # add extension here
+        if @data["extension"].present?
+          tags << tag.span(class: "extension") { @data["extension"] }
+        end
+      else
+        tags << tag.span { Rinku.auto_link(h(value)).html_safe }
+      end
+      safe_join tags
     end
 
     def link