Support delete with no bean #1926

Merged
lunny merged 4 commits from lunny/delete into master 5 months ago
lunny commented 6 months ago
Owner

Now you can use delete like this:

orm.Table("my_table").Where("id=?",1).Delete()
Now you can use delete like this: ``` orm.Table("my_table").Where("id=?",1).Delete() ```
lunny added the
kind/enhancement
label 6 months ago
lunny added 1 commit 6 months ago
07444ffe27
support delete with no bean
lunny force-pushed lunny/delete from 07444ffe27 to 286a929433 6 months ago
lunny added 1 commit 6 months ago
4552e26d7b
Update README
lunny added 1 commit 6 months ago
c4f8c17d71
Fix build
lunny added 1 commit 6 months ago
515a0ff4ce
Fix test
lunny added this to the 1.2.0 milestone 6 months ago
lunny added the
kind/breaking
label 6 months ago

since .Delete(&User{ID: 1}, &User{ID: 2}, ...) is now also valide, should it be checked or supported?

since `.Delete(&User{ID: 1}, &User{ID: 2}, ...)` is now also valide, should it be checked or supported?
Poster
Owner

since .Delete(&User{ID: 1}, &User{ID: 2}, ...) is now also valide, should it be checked or supported?

Should we support deleted multiple object here?

> since `.Delete(&User{ID: 1}, &User{ID: 2}, ...)` is now also valide, should it be checked or supported? Should we support deleted multiple object here?
Poster
Owner

since .Delete(&User{ID: 1}, &User{ID: 2}, ...) is now also valide, should it be checked or supported?

That could be another PR.

> since `.Delete(&User{ID: 1}, &User{ID: 2}, ...)` is now also valide, should it be checked or supported? That could be another PR.
lunny merged commit 8f64a78cd4 into master 5 months ago
lunny referenced this issue from a commit 5 months ago
lunny deleted branch lunny/delete 5 months ago
continuous-integration/drone/push Build is passing
continuous-integration/drone/pr Build is passing
The pull request has been merged as 8f64a78cd4.
Sign in to join this conversation.
Loading…
There is no content yet.