Fix id mess #9

New issue

Open

opened 2026-05-20 00:06:39 +02:00 by DestinyofYeet · 0 comments

DestinyofYeet commented 2026-05-20 00:06:39 +02:00

Owner

Copy link

• Model.get_id() currently controls wether the model is actually saved in the database or not.
• The SqliteStrategy needs a column named id when saving, but this is not checked anywhere.
• It is also expected that the id gets returned

- `Model.get_id()` currently controls wether the model is actually saved in the database or not. - [The SqliteStrategy needs a column named `id` when saving](https://code.ole.blue/DestinyofYeet/django-rs/src/commit/5bd92df27e7151b366f9b98618e97676a84d0f2c/src/server/database_strategy/default_strategies/sqlite_strategy.rs#L304), but this is not checked anywhere. - [It is also expected that the `id` gets returned](https://code.ole.blue/DestinyofYeet/django-rs/src/commit/5bd92df27e7151b366f9b98618e97676a84d0f2c/src/server/database_strategy/default_strategies/sqlite_strategy.rs#L322)

Sign in to join this conversation.

No Branch/Tag specified

Branches Tags

main

develop

fix/proc-macro

feat/parsing-rewrite

feat/serde_derive

v0.1

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

DestinyofYeet/django-rs#9

No description provided.