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.

Sign up for GitHub

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

Jump to bottom

Add initial implementation #2

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Add initial implementation #2

wants to merge 4 commits into from

Conversation

nechaido
Copy link
Member

@nechaido nechaido commented May 17, 2021 •
edited
Loading

Depends on: #1

Closes: metarhia/metaschema#298

Examples:
PlantUML:
image

Mermaid:
image

@nechaido nechaido requested a review from tshemsedinov May 17, 2021 11:14
@nechaido
Copy link
Member Author

@tshemsedinov there is an issue with enum types.

It would be usefull to somewhat name enums, or use enums' key from ./types.js as a name.
This will allow me to display it as a property type.

@nechaido
Copy link
Member Author

@tshemsedinov it's quite odd to check indexes to get many-to-many relations. See last commit.

@nechaido nechaido force-pushed the add-initial-implementation branch from bba31a6 to aba1ecd Compare May 18, 2021 07:51
This was referenced May 19, 2021
Open
Closed
@nechaido
Copy link
Member Author

Ping @tshemsedinov PTAL.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tshemsedinov tshemsedinov Awaiting requested review from tshemsedinov

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

ER generator
1 participant
@nechaido