Skip to content

Conversation

@maricavor
Copy link
Contributor

@maricavor maricavor commented May 21, 2025

Close #2792

Refactoring and Consolidation of Domain Save Operations

  • Removed multiple intermediate domain.save(validate: false) calls from the following interaction classes: PostSetProcess, PrepareDomain, SetStatus
  • Moved the domain save operation to a single place in SetForceDelete, wrapped in a database transaction. This ensures all changes are saved atomically and only one PaperTrail version is created.
  • Improved Force Delete Status Note Handling
  • Refactored how force delete status notes are set for domains
  • Enhanced Email-Related Force Delete Processing
  • Refactored Domains::ForceDeleteEmail::Base
  • Improved readability and structure by introducing clearer helper methods

@vohmar vohmar requested a review from OlegPhenomenon May 29, 2025 13:49
@vohmar vohmar merged commit 4d536a9 into master May 30, 2025
1 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Admin: duplicate domain and contact history records

4 participants