Skip to content

feat: implement CursorTrait for SelectFour#3058

Draft
zefr0x wants to merge 1 commit into
SeaQL:masterfrom
zefr0x:cursor_by_for_select_four
Draft

feat: implement CursorTrait for SelectFour#3058
zefr0x wants to merge 1 commit into
SeaQL:masterfrom
zefr0x:cursor_by_for_select_four

Conversation

@zefr0x

@zefr0x zefr0x commented May 6, 2026

Copy link
Copy Markdown

PR Info

  • Closes

New Features

  • Implement cursor_by() for SelectFour

@zefr0x zefr0x marked this pull request as draft May 6, 2026 17:03
@zefr0x

zefr0x commented May 6, 2026

Copy link
Copy Markdown
Author

We still need tests for this, so it will stay as a draft PR.

I don't know why it wasn't implemented or if my implementation is valid. I just copied SelectThree's impl and tuned it to handle four tables. Used it in my project and it seem to work.

@zefr0x zefr0x force-pushed the cursor_by_for_select_four branch from d5aca7b to 57496e8 Compare June 1, 2026 18:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant