-
-
Notifications
You must be signed in to change notification settings - Fork 465
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.
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
SelectTwo cursor by, cursorTrait revamp #1826
Conversation
tests/upsert_tests.rs
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a very interesting test case. But 1) please add, not change and 2) this is not relevant to the current PR?
#[smol_potat::test] | ||
async fn first_2_before_10_also_related_select() -> Result<(), DbErr> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These tests are cool! Would appreciate a few integration tests (in sqlite/mysql/postgres)
…and primary key from other table
} | ||
|
||
impl<S> Cursor<S> | ||
where | ||
S: SelectorTrait, | ||
{ | ||
/// Initialize a cursor | ||
pub fn new<C>(query: SelectStatement, table: DynIden, order_columns: C) -> Self | ||
pub fn new<C>( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note to self: new with
This has also been released in |
PR Info
CursorTrait
forSelectTwo
#1805New Features
Bug Fixes
Breaking Changes
Changes
CursorTrait
behaviour, can be phased out in future