Skip to content
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

Ignore Composite Foreign Keys. #1299

Merged
merged 1 commit into from
Sep 4, 2023

Conversation

paulo-raca
Copy link
Contributor

Boiler doesn't support composite FKs -- And this is OK.

However if there is one on the database, it should ignore it and generate valid code.

This addresses #1298

@paulo-raca paulo-raca force-pushed the ignore-composite-fk branch from 0a930b2 to f03a54d Compare August 30, 2023 19:28
@paulo-raca
Copy link
Contributor Author

paulo-raca commented Aug 30, 2023

I've updated it with tests for Postgres+MySQL+SQLite3+MS-SQL.

@paulo-raca paulo-raca force-pushed the ignore-composite-fk branch 2 times, most recently from 84d21c4 to 9d99317 Compare August 30, 2023 21:12
Boiler doesn't support composite FKs -- And this is OK.

However if there is one on the database, it should ignore it and generate valid code.
@paulo-raca paulo-raca force-pushed the ignore-composite-fk branch from 9d99317 to 2cb4f9b Compare August 30, 2023 21:21
@stephenafamo stephenafamo merged commit 96378fa into volatiletech:master Sep 4, 2023
2 checks passed
@adsteel
Copy link

adsteel commented Jan 9, 2024

@stephenafamo This is pretty exciting and currently blocking us. What's the chance we could get a release cut that includes this fix? Happy to help with that if there's anything I/we can do, too.

@stephenafamo
Copy link
Collaborator

The latest release should already include it. Are you still having issues?

@adsteel
Copy link

adsteel commented Jan 9, 2024

We are on 4.15.0 and having issues, but I think this merged a few days after that release shipped.

@stephenafamo
Copy link
Collaborator

That's true, I got things mixed up, I haven't done a release in a while.

I'll tag a new version.

@adsteel
Copy link

adsteel commented Jan 9, 2024

Wonderful, thank you!

@adsteel
Copy link

adsteel commented Jan 16, 2024

Anything I can do to help get a new release out?

@stephenafamo
Copy link
Collaborator

Released now. Apologies for the delay @adsteel

@adsteel
Copy link

adsteel commented Jan 16, 2024

@stephenafamo no problem at all. This is exciting, thank you!

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