Skip to content

Conversation

@Anurag9977
Copy link

@Anurag9977 Anurag9977 commented Jan 17, 2026

  • Updated the TransactionalAdapter interface and the TransactionalOptionsAdapterFactory type with optional properties custom provider tokens and custom providers respectively.
  • Updated the ClsPluginTransactional class to include extraProviderTokens into the TransactionalAdapterOptions provider.
  • Updated CustomAdapter docs.

relates to #444

@Anurag9977 Anurag9977 force-pushed the feat/transactional_allows-injecting-custom-providers branch from dd5d78f to 33a0a21 Compare January 18, 2026 08:57
@Papooch
Copy link
Owner

Papooch commented Jan 18, 2026

Hi, thank you for the PR. I made some maintenance over the weekend, so could you please rebase your branch on top of the current main?

Also, before we merge this feature, please add some tests that cover it. If you aren't sure how to approach it, let me know and I'll try to give you some guidance.

@Anurag9977
Copy link
Author

Hi, Sure. I'll rebase and update the PR.
Regarding tests, I believe we need to add a custom transactional adapter test but I would appreciate some pointers or maybe any existing test I can look at. That would be helpful.

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.

2 participants