Skip to content

fix: remove catch from handler in transaction#444

Open
charsleysa wants to merge 3 commits intoporsager:masterfrom
charsleysa:fix-errors
Open

fix: remove catch from handler in transaction#444
charsleysa wants to merge 3 commits intoporsager:masterfrom
charsleysa:fix-errors

Conversation

@charsleysa
Copy link
Copy Markdown

Fixes an issue where catching an error inside a transaction and running custom error logic doesn't work due to a hidden catch and rethrow occurring.

@skarbovskiy
Copy link
Copy Markdown

@porsager Are there any plans to merge this?

@mattbishop
Copy link
Copy Markdown

@charsleysa why does this PR have the docker file changes?

@charsleysa
Copy link
Copy Markdown
Author

@mattbishop those were config files added for working in VSCode isolated workspaces. I can remove them if it's blocking the PR.

@isaac-mcfadyen
Copy link
Copy Markdown

Any thoughts on when this can be merged?

@porsager porsager force-pushed the master branch 2 times, most recently from 4fd011e to a92f470 Compare May 20, 2025 23:40
@porsager porsager force-pushed the master branch 2 times, most recently from 4a0fe34 to 3a43815 Compare November 12, 2025 04:40
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.

4 participants