Skip to content

Add reference_options struct to contain options#14

Draft
sc68cal wants to merge 1 commit intojetzig-framework:mainfrom
sc68cal:on_delete
Draft

Add reference_options struct to contain options#14
sc68cal wants to merge 1 commit intojetzig-framework:mainfrom
sc68cal:on_delete

Conversation

@sc68cal
Copy link
Contributor

@sc68cal sc68cal commented Mar 21, 2025

This allows the addition of ON DELETE CASCADE and other options

Fixes #13

This allows the addition of ON DELETE CASCADE and other options

Fixes jetzig-framework#13
@sc68cal
Copy link
Contributor Author

sc68cal commented Mar 21, 2025

Will add tests in next session

@sc68cal
Copy link
Contributor Author

sc68cal commented Mar 21, 2025

Originally I wanted to change the jetquery.schema.Column.Reference to make it a struct, something like

pub const Reference = struct {
    table: []const u8 = null,
    column: []const u8 = null,
    on_delete: ?DeleteOptions = null,
};

But my zig-fu isn't strong enough to figure out how to turn it into a struct while still supporting the original array initialization syntax (if that's even possible in Zig), and I didn't want to make a breaking API change.

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.

Expand Reference to include ON DELETE options

1 participant