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

Migrate CrudOperations Macros to Use QueryBuilder #68

Open
TheRustifyer opened this issue Feb 11, 2025 · 0 comments
Open

Migrate CrudOperations Macros to Use QueryBuilder #68

TheRustifyer opened this issue Feb 11, 2025 · 0 comments

Comments

@TheRustifyer
Copy link
Member

Description:

To unify the generation of user queries and enhance maintainability, we should transition all query string constructions within the CrudOperations macros to utilize the QueryBuilder. This change will centralize SQL statement generation, making the macros more straightforward and the statements immediately testable.

Dependencies:

Tasks:

  • Identify all instances within the CrudOperations macros where SQL query strings are manually constructed.
  • Refactor these macros to leverage the QueryBuilder for SQL statement generation.
  • Ensure that the refactored macros maintain their existing functionality and performance characteristics.
  • Write or update unit tests to validate that the macros produce correct and efficient SQL statements using the QueryBuilder.

Expected Outcome:

A cohesive and maintainable codebase where CrudOperations macros utilize the QueryBuilder for SQL generation, leading to improved control over generated statements and easier testing.

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

No branches or pull requests

1 participant