-
Notifications
You must be signed in to change notification settings - Fork 44
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
Rotorsoft/issue5293 - Create a shell script to read the current database schema and generate a mermaid ERD in the wiki space #5294
Conversation
…i after database migrations Fixes #5293, gitignore .vscode to allow local configs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is awesome! Super cool idea elegantly implemented.
My only big-picture concern is that it's such an enormous ERD, it's difficult to view or navigate visually. Curious if you have any advice, or what you've been using to visually navigate the generated ERD.
Nice. Looks like the script assumes that Postgres is running directly on the host machine? For those of us who run Postgres in Docker (Compose) it would be nice to have a script (or documentation including the script) that can work in that context. |
I also run PG in docker, it should work unless you are using a different PG config. |
I normally use the mermaid live editor - https://mermaid.live , but the Github integration should be good enough. There is also the Miro and VSCode integrations... https://mermaid.js.org/ecosystem/integrations-community.html |
Do you have |
yes |
Looks like a minor merge conflict, otherwise looks good from a documentation perspective, thanks for the swift changes. I wasn't having great luck with an ERD this size using Matt Bierner's Markdown Mermaid Support extension for VSCode, but I'll give the official Mermaid extension a shot. Either way, a minor consideration. |
Link to Issue
Closes: #5293
Description of Changes
Test Plan
Other Considerations
db-all
script if necessary