Skip to content

Enhance error handling and logging in context methods#640

Merged
StuartFerguson merged 2 commits intomasterfrom
bug/#637_&_#638_failed_event_loading
Jan 9, 2025
Merged

Enhance error handling and logging in context methods#640
StuartFerguson merged 2 commits intomasterfrom
bug/#637_&_#638_failed_event_loading

Conversation

@StuartFerguson
Copy link
Copy Markdown
Member

  • Updated SaveChangesWithDuplicateHandling to return a Result object and log unique constraint exceptions.
  • Refactored EstateReportingRepository to use the new save method for better handling of unique constraint violations.
  • Modified DomainEventController to use Task<Result> for event handling, improving error reporting for failed tasks.

- Updated `SaveChangesWithDuplicateHandling` to return a `Result` object and log unique constraint exceptions.
- Refactored `EstateReportingRepository` to use the new save method for better handling of unique constraint violations.
- Modified `DomainEventController` to use `Task<Result>` for event handling, improving error reporting for failed tasks.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Failed Processing Event - MerchantFeeAddedPendingSettlementEvent Failed Processing Event - SettledMerchantFeeAddedToTransactionEvent

1 participant