-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
[Feature] Add Documentation Menu with Markdown Export for Database Documentation #229
[Feature] Add Documentation Menu with Markdown Export for Database Documentation #229
Conversation
@LuigimonSoft is attempting to deploy a commit to the dottle's projects Team on Vercel. A member of the Team first needs to authorize it. |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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 a great feature. Thanks for the contribution.
Other than the comments, please move the dropdown to export as readme, I don't want people to think it's drawdb docs.
That should be pretty much it but I'll take a closer look at night
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.
Misclicked the approved sorry
Than you for you feedback |
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.
Just had a closer look. Idk why I just thought of it but you can also document the enums and types.
Thank you
src/utils/exportAs/documentation.js
Outdated
`## Introduction\n\n## Database type\n\n- **Database system:** `+ | ||
`${databases[obj.database].name}\n## Table structure\n\n${documentationEntities}`+ | ||
`\n## Relationships\n\n${documentationRelationships}\n` + | ||
`${databases[obj.database].hasTypes && obj.types.length > 0 ? `## Types\n\n` + documentationTypes`\n\n` : "" }` + |
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.
There's a missing plus sign between documentationTypes and \n\n that ends up throwing an error. Please test on postgres and add types before trying to export
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.
Looks good, thank you
What
This pull request introduces a new "Documentation" menu item to the application, allowing users to export database documentation in Markdown format. The documentation is structured in a data dictionary format, including details such as table names, columns, constraints, indexes, and relationships.
Why
The purpose of this feature is to enhance the user experience by providing a streamlined way to generate and export database documentation in Markdown format. This allows users to easily share, review, and manage the database schema in a widely accepted and readable format.
How
Add Documentation Menu:
Generate Markdown Documentation:
Export Functionality:
close #228