Unarchive the provider themselves too

apps/accounts/models/provider_request.py

@@ -231,6 +231,9 @@ def approve(self) -> Hostingprovider:
         hp.request = self
         hp.created_by = self.created_by
 
+        # if we have approved a submission from a provider
+        hp.archived = False
+
         # set services (https://django-taggit.readthedocs.io/en/latest/api.html)
         hp.services.set(list(self.services.all()))
         hp.staff_labels.add("up-to-date")

apps/accounts/tests/test_provider_request.py

@@ -576,6 +576,7 @@ def test_approve_updates_existing_provider(hosting_provider_with_sample_user):
     # provider is visible by default
     # appropriate tag is added
     assert hp.showonwebsite is True
+    assert hp.archived is False
     assert "up-to-date" in hp.staff_labels.slugs()
     # "other-none" is the label condition check for
     # when someone is just trying to get a site marked