From 23f5da4a9e07c542fb5f9188174cfbd1257ab3f1 Mon Sep 17 00:00:00 2001
From: StuartFerguson <stuart_ferguson1development@outlook.com>
Date: Sun, 17 May 2026 06:22:11 +0100
Subject: [PATCH] Refactor exception handling and clean up interface

Improved exception handling in TransactionProcessorClient by adding contextual information and removed an unnecessary semicolon. Cleaned up ITransactionProcessorReadModelRepository by deleting obsolete commented-out voucher methods.
---
 TransactionProcessor.Client/TransactionProcessorClient.cs   | 1 -
 .../ITransactionProcessorReadModelRepository.cs             | 6 ------
 2 files changed, 7 deletions(-)

diff --git a/TransactionProcessor.Client/TransactionProcessorClient.cs b/TransactionProcessor.Client/TransactionProcessorClient.cs
index 1387ce71..e498f5f8 100644
--- a/TransactionProcessor.Client/TransactionProcessorClient.cs
+++ b/TransactionProcessor.Client/TransactionProcessorClient.cs
@@ -714,7 +714,6 @@ public async Task<Result> CreateEstateUser(String accessToken,
                 return ResultHelpers.CreateFailure(result);
 
             return result;
-            ;
         }
         catch (Exception ex) {
             // An exception has occurred, add some additional information to the message
diff --git a/TransactionProcessor.Repository/ITransactionProcessorReadModelRepository.cs b/TransactionProcessor.Repository/ITransactionProcessorReadModelRepository.cs
index eaab65fc..875d5974 100644
--- a/TransactionProcessor.Repository/ITransactionProcessorReadModelRepository.cs
+++ b/TransactionProcessor.Repository/ITransactionProcessorReadModelRepository.cs
@@ -218,12 +218,6 @@ Task<Result> UpdateTransactionAuthorisation(TransactionDomainEvents.TransactionA
         Task<Result> UpdateTransactionAuthorisation(TransactionDomainEvents.TransactionDeclinedByOperatorEvent domainEvent,
                                                     CancellationToken cancellationToken);
 
-        //Task<Result> UpdateVoucherIssueDetails(VoucherDomainEvents.VoucherIssuedEvent domainEvent,
-        //                                       CancellationToken cancellationToken);
-
-        //Task<Result> UpdateVoucherRedemptionDetails(VoucherDomainEvents.VoucherFullyRedeemedEvent domainEvent,
-        //                                            CancellationToken cancellationToken);
-
         Task<Result> RemoveOperatorFromMerchant(MerchantDomainEvents.OperatorRemovedFromMerchantEvent domainEvent,
                                                 CancellationToken cancellationToken);