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

Issue 219 #237

Merged
merged 1 commit into from
Oct 9, 2023
Merged

Issue 219 #237

merged 1 commit into from
Oct 9, 2023

Conversation

keonik
Copy link
Owner

@keonik keonik commented Aug 22, 2023

closes #219 by combining relationship names from both models into one connection string.

…hip names to and from end model, then skipping end model. Fix issue with misrepresented one/one ot zero/one relationships
@keonik keonik changed the base branch from main to alpha August 22, 2023 14:34
@naitaku
Copy link

naitaku commented Oct 6, 2023

@keonik Could you please consider making an alpha release with this fix?

@keonik
Copy link
Owner Author

keonik commented Oct 9, 2023

I will push as is but the relationship for many to x is messed up for sure. I haven't had time to work on it for a while. Meaning alpha will not be published for a bit

@keonik keonik merged commit f1aa362 into alpha Oct 9, 2023
@github-actions
Copy link

github-actions bot commented Oct 9, 2023

🎉 This PR is included in version 1.12.0-alpha.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unnecessary Many-to-Many Relation Lines in ER Diagram
2 participants