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.

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

[Feature] Migration from russqlite to diesel #327

Closed
CommanderStorm opened this issue Dec 17, 2022 · 0 comments · Fixed by #334
Closed

[Feature] Migration from russqlite to diesel #327

CommanderStorm opened this issue Dec 17, 2022 · 0 comments · Fixed by #334
Assignees
@CommanderStorm
Labels
feature New feature or request server Related to the backend/server

Comments

@CommanderStorm
Copy link
Member

CommanderStorm commented Dec 17, 2022 •
edited

Is your feature request related to a problem? Please describe.
The API from russqlite not great to work with and likely error prawn.
I think it is time to switch to a real ORM.

Describe the solution you'd like
I would like to exchange all usages of russqlite with the respective usecases from diesel

Describe alternatives you've considered
Sticking with russqlite. While being an option, manually constructing queries and extracting from them increases the likelyhood of errors.

Additional context
This is becoming an issue in #324, as there more SQL is needed.
@CommanderStorm CommanderStorm added feature New feature or request server Related to the backend/server labels Dec 17, 2022
@CommanderStorm CommanderStorm changed the title [Feature] Migration from russqlite to diesel [Feature] Migration from [russqlite](https://docs.rs/rusqlite/latest/rusqlite/) to [diesel](https://diesel.rs/) Dec 17, 2022
@CommanderStorm CommanderStorm changed the title [Feature] Migration from [russqlite](https://docs.rs/rusqlite/latest/rusqlite/) to [diesel](https://diesel.rs/) [Feature] Migration from russqlite to diesel Dec 17, 2022
@CommanderStorm CommanderStorm mentioned this issue Dec 22, 2022
Merged
@CommanderStorm CommanderStorm self-assigned this Dec 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@CommanderStorm CommanderStorm

Labels
feature New feature or request server Related to the backend/server
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Cal-1] migration from russql to Diesel TUM-Dev/NavigaTUM
1 participant
@CommanderStorm