Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Dapper Provider #4998

Merged
merged 2 commits into from
Feb 25, 2024
Merged

Fix Dapper Provider #4998

merged 2 commits into from
Feb 25, 2024

Conversation

sfmskywalker
Copy link
Member

This PR fixes a number of issues with the Dapper provider, specifically when using it against a SQL Server database.

The update includes a significant enhancement to the Dapper module. Improvements were made to support SQL Server and enhanced SQL query pagination. Additionally, table column size was increased for various fields to handle more extensive data. Minor refactoring was also done to improve code readability.
The previous setting had SQL Server usage in Elsa server enabled by default. This commit changes the useSqlServer constant in Elsa.Server.Web to false, effectively switching SQL Server off unless manually activated.
@sfmskywalker sfmskywalker requested review from a team February 25, 2024 19:20
@sfmskywalker sfmskywalker added bug Something isn't working elsa 3 This issue is specific to Elsa 3 labels Feb 25, 2024
@sfmskywalker sfmskywalker merged commit 330dc1a into main Feb 25, 2024
6 checks passed
@sfmskywalker sfmskywalker deleted the fix/dapper-provider branch February 25, 2024 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working elsa 3 This issue is specific to Elsa 3
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants