Skip to content

[Database] Add SQLite support#12

Merged
techmahedy merged 14 commits intodoppar:2.xfrom
rrr63:refactor-database-drivers
Oct 15, 2025
Merged

[Database] Add SQLite support#12
techmahedy merged 14 commits intodoppar:2.xfrom
rrr63:refactor-database-drivers

Conversation

@rrr63
Copy link
Copy Markdown
Member

@rrr63 rrr63 commented Oct 6, 2025

Refactor database grammar to allow multiple drivers.
Add SQLite driver

#11

@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 6, 2025

Thanks for submitting a PR!

Note that draft PR's are not reviewed. If you would like a review, please mark your pull request as ready for review in the GitHub user interface.

Pull requests that are abandoned in draft may be closed due to inactivity.

@techmahedy
Copy link
Copy Markdown
Member

@rrr63, great to see that you started, but remember we have almost 2 months in our hands for our first release, so take your time.

@rrr63
Copy link
Copy Markdown
Member Author

rrr63 commented Oct 6, 2025

@techmahedy Yes for sure, it is just the first push, not ready yet.
Btw, I think you should open the new doc to PR, i saw some things to change there

@techmahedy
Copy link
Copy Markdown
Member

@rrr63, of course, you can share about this "I think you should open the new doc to PR, i saw some things to change there".

@rrr63
Copy link
Copy Markdown
Member Author

rrr63 commented Oct 6, 2025

@techmahedy I can't open PR because the doc : https://github.com/doppar/docs does not seem to be the current website

@techmahedy
Copy link
Copy Markdown
Member

@rrr63 Exactly, this is not the current one, I will update it

@techmahedy
Copy link
Copy Markdown
Member

@rrr63 https://github.com/doppar/docs Updated, you can check this.

@rrr63
Copy link
Copy Markdown
Member Author

rrr63 commented Oct 7, 2025

@techmahedy perfect, what is the doc engine to reproduce it locally ?
Perhaps, I think you should prepare project with more contributing guides to attract more contributors.
I can help you by this way if you want

@techmahedy
Copy link
Copy Markdown
Member

@rrr63 if you help me, it will be very beneficial for doppar, you are a most senior person that's why i added you to doppar team to make doppar more professional. Very glad to hear that. You can share your mail so that i can add you in my slack channel.

@rrr63
Copy link
Copy Markdown
Member Author

rrr63 commented Oct 7, 2025

@techmahedy I sent you an email

@rrr63 rrr63 marked this pull request as ready for review October 8, 2025 13:57
@rrr63 rrr63 force-pushed the refactor-database-drivers branch from d77decf to 03c6fe6 Compare October 8, 2025 14:01
@rrr63
Copy link
Copy Markdown
Member Author

rrr63 commented Oct 8, 2025

@techmahedy This PR is open to code review

@rrr63 rrr63 added the feat new feature label Oct 8, 2025
@techmahedy
Copy link
Copy Markdown
Member

Ok @rrr63

@techmahedy
Copy link
Copy Markdown
Member

@rrr63 my last commit: "fix: mysql enum values and adding grammar-based query in eloquent builder:"

Everything should work now. Please check everything and let me know. I tried to fix everything.

@techmahedy techmahedy merged commit 98041dd into doppar:2.x Oct 15, 2025
18 checks passed
@rrr63 rrr63 linked an issue Oct 15, 2025 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Database] Making database works with PSQL and SQLITE

3 participants