Skip to content

Remove sqlRow now! #3

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.

Sign up for GitHub

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

Jump to bottom
Closed
2 tasks done
parsa-mostafaie opened this issue Jul 12, 2024 · 1 comment
Closed
2 tasks done

Remove sqlRow now! #3

parsa-mostafaie opened this issue Jul 12, 2024 · 1 comment
Assignees
@parsa-mostafaie
Labels
enhancement New feature or request good first issue Good for newcomers invalid This doesn't seem right

Comments

@parsa-mostafaie
Copy link
Owner

parsa-mostafaie commented Jul 12, 2024 •
edited
Loading

We know the sqlRow class is writed to used until the time that pluslib add support for models, now the models are supported so sqlRow can be removed from the library (framework 😝).

  • Use models instead of sqlRow in example projects
  • Then remove it from the library
@parsa-mostafaie parsa-mostafaie added enhancement New feature or request good first issue Good for newcomers invalid This doesn't seem right labels Jul 12, 2024
@parsa-mostafaie parsa-mostafaie self-assigned this Jul 12, 2024
@parsa-mostafaie parsa-mostafaie changed the title Don't use sqlRow now! Remove use sqlRow now! Jul 12, 2024
@parsa-mostafaie parsa-mostafaie changed the title Remove use sqlRow now! Remove sqlRow now! Jul 12, 2024
@parsa-mostafaie parsa-mostafaie pinned this issue Jul 13, 2024
@parsa-mostafaie
Copy link
Owner Author

We are one step closer to solving this! (In 7715922)

How?

The classes: sqlRow and sql_abcol are now deprecated!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@parsa-mostafaie parsa-mostafaie

Labels
enhancement New feature or request good first issue Good for newcomers invalid This doesn't seem right
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@parsa-mostafaie