drawdb-io / drawdb

Free, simple, and intuitive online database diagram editor and SQL generator.

https://drawdb.app

GNU Affero General Public License v3.0

22.75k stars 1.59k forks source link

feat:Added support for exporting SQL as OracleDB SQL #192

Open Kingpin98k opened 3 months ago

Kingpin98k commented 3 months ago

Oracle Database SQL Export Functionality in DrawDB

Summary of Current Implementation

Export to Oracle SQL Files:
Successfully implemented the feature to convert database schemas into Oracle Database compatible SQL files. ![welcomeTab](https://github.com/user-attachments/assets/43f5f6ca-b975-4fe1-a319-a20c9b06313c) ![image](https://github.com/user-attachments/assets/79c42996-f332-413f-830b-f16846cf8aa0)
Import Functionality:

Currently under evaluation for feasibility and development planning.

Request for Guidance

Merge Approval: Requesting permission to merge the export feature into the main repository.
Next Steps: Seeking direction on whether to proceed with the import functionality or prioritize other tasks.

I am eager to align this project with our overall objectives and user needs. Please provide feedback and any specific considerations for the next steps.

vercel[bot] commented 3 months ago

@Kingpin98k is attempting to deploy a commit to the dottle's projects Team on Vercel.

A member of the Team first needs to authorize it.

1ilit commented 3 months ago

Cool stuff man! I'm really excited about this

Kingpin98k commented 3 months ago

Cool stuff man! I'm really excited about this

Thanks brother 🙂, tell me if any changes needs to be done.

1ilit commented 3 months ago

Hey! This is great and really would love to merge but here's my dilemma. I don't think we can claim that oracle is fully supported because the import will be missing for a while. I don't know if we should disable import for oracle and have it incomplete or support is as another export option only for generic diagrams.

Kingpin98k commented 3 months ago

Yeah I am kinda busy for now but will sure work on the import also, so till then you can leave it be and merge the feature once import is done 🙂👍

gvenzl commented 3 months ago

Hey @Kingpin98k, this is amazing!

Thanks a lot for doing this PR, I'm really excited about it! Please feel free to reach out to me if I can be of any help!

Kingpin98k commented 3 months ago

Hey @Kingpin98k, this is amazing!

Thanks a lot for doing this PR, I'm really excited about it! Please feel free to reach out to me if I can be of any help!

Hey @gvenzl thanks for the insights, I will make sure to incorporate these changes in my next PR and will definitely reach out to you if I need any assistance.