Skip to content

Add support for many to many relation#211

Merged
topclaudy merged 6 commits into
topclaudy:masterfrom
akalongman:feature/support-many-to-many
Apr 22, 2026
Merged

Add support for many to many relation#211
topclaudy merged 6 commits into
topclaudy:masterfrom
akalongman:feature/support-many-to-many

Conversation

@akalongman

Copy link
Copy Markdown
Contributor

Issue #198

@akalongman

Copy link
Copy Markdown
Contributor Author

@topclaudy Could this MR be reviewed please?

@topclaudy

topclaudy commented Mar 19, 2026

Copy link
Copy Markdown
Owner

Thanks @akalongman. Great work on this! It's a solid implementation. Out of curiosity, was any of this AI-assisted? Totally fine if so, I'm just curious about the workflow as it would also help me tailor the review.

@akalongman

Copy link
Copy Markdown
Contributor Author

@topclaudy yes, I actively use Claude Code as a helper

@topclaudy

Copy link
Copy Markdown
Owner

@akalongman It looks like ~75% of the tests are failing. Could you take a look when you get a chance?

@akalongman

Copy link
Copy Markdown
Contributor Author

Oh, I did not test it for older laravels than v12 😞 I will take a look

@akalongman

akalongman commented Mar 23, 2026

Copy link
Copy Markdown
Contributor Author

@topclaudy I propose releasing version 3.0 and discontinuing support for L11 and earlier versions. These versions have reached their official end-of-life and concluded their security update cycles. https://endoflife.date/laravel

@akalongman

Copy link
Copy Markdown
Contributor Author

Any feedback about it?

@npostman

Copy link
Copy Markdown

+1. Great addition and already implemented in a dev project and working nicely.
@topclaudy Any chance you can have a look at merging this (and thus dropping < L12 support)?

@topclaudy

Copy link
Copy Markdown
Owner

I’ll get to it later this week.

@topclaudy topclaudy merged commit 0cc503f into topclaudy:master Apr 22, 2026
7 of 8 checks passed
@topclaudy

Copy link
Copy Markdown
Owner

Released in 3.0.0. Thanks @akalongman

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.

3 participants