cisagov / manage.get.gov

A Django-based domain name registrar that interfaces with an EPP registry
https://get.gov
Other
60 stars 19 forks source link

Update delete domain process to remove hosts and/or contacts #2823

Open vickyszuchin opened 2 months ago

vickyszuchin commented 2 months ago

Issue description

When removing a domain from the registry, the registrar threw the error: "Error deleting this Domain: This subdomain is being used as a hostname on another domain."

We should go ahead and automatically remove the subdomain nameservers first as part of our delete process. Right now on delete we are just removing the domain from the registry. We do not remove the hosts or contacts. We originally didn’t add in a check on the subdomain thing because we were not storing nameservers. Now we are are storing nameservers so it is not overly complex for us to just check for where it is in use and auto remove if it is just in use on the current domain.

Additionally, if the subdomain is in use on another domain we should have a friendly error that shows for you saying “Can’t delete because a subdomain of this domain is in use on: xyz.gov, abc.gov..."

Acceptance criteria

Additional context

Slack thread

Links to other issues

No response

vickyszuchin commented 2 months ago

@abroddrick a reminder to refine this ticket. You can remove the "refinement" label afterwards.

h-m-f-t commented 1 month ago

@abroddrick, domains in Deleted still have expiration dates, which feels wrong. Since we're updating the delete domain process here, would it be appropriate to add something like "remove the expiration date from a domain being put in Deleted" here? (We could create a different issue to expire the expiration from currently Deleted domains.)

abroddrick commented 2 weeks ago

Hey just for visibility, @h-m-f-t, @PaulKuykendall and @katypies, Liz and I moved this one out of the sprint to make room for the pod work (as well as one of the housekeeping ones).

h-m-f-t commented 2 weeks ago

Fair, thanks for the heads up!

h-m-f-t commented 2 weeks ago

Adding refinement to get an answer to my question about expiration above. @abroddrick @lizpearl