Add exists method to PaginatorTrait with tests #2623
Conversation
|
Thank you! I'll review this a bit later. TODO for myself: also merge this in If you're ok with using your own unreleased fork of 1.1.x until 2.0.0 comes out, you can change the base of the PR to |
Can this be released as a patch 1.1.x release? I think it should be fine since it does not have any breaking changes. That way I can continue to use the crates.io version. |
|
Yeah, I assumed that you may want that. I'm not the one making (the decisions about) releases, but @tyt2y3 will probably do this |
| @@ -69,7 +70,7 @@ pub async fn init_1(ctx: &TestContext, link: bool) { | |||
| total: Set(10.into()), | |||
| bakery_id: Set(42), | |||
| customer_id: Set(11), | |||
| placed_at: Set(DateTime::UNIX_EPOCH), | |||
| placed_at: Set(ChronoDateTime::UNIX_EPOCH.naive_utc()), | |||
There was a problem hiding this comment.
did this because this was throwing a deprecation warning
Introduce an
existsmethod to thePaginatorTraitto efficiently check for record existence.Fixes #730.