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.

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

[Chore] Remove support for sqlite #669

Merged
merged 7 commits into from Aug 2, 2023
Merged

Conversation

alexjustesen
Copy link
Owner

Description

This PR removes SQLite as the default database option, steps to use SQLite can be referenced from the Laravel docs should that still want to be used.

Changelog

Removed

  • SQLite as the default database

@alexjustesen alexjustesen added the 🧹 chore Nothing fancy, just needs to get done label Aug 2, 2023
@alexjustesen alexjustesen self-assigned this Aug 2, 2023
@alexjustesen alexjustesen changed the title Remove support for sqlite [Chore] Remove support for sqlite Aug 2, 2023
@alexjustesen alexjustesen merged commit 3024123 into main Aug 2, 2023
2 checks passed
@alexjustesen alexjustesen deleted the remove-support-for-sqlite branch August 2, 2023 01:18
@AleksCee
Copy link

Without a way to migrate with the braking changes?

@luckman212
Copy link

So if we don't have an InfluxDB set up, do we now need to stand one up? No more single-container instance?

@alexdelprete
Copy link
Contributor

So if we don't have an InfluxDB set up, do we now need to stand one up? No more single-container instance?

you need mysql/mariadb or postgresql as main db.

Influxdb is optional, to export data for better analysis/graphs, etc. it's not mandatory.

@alexjustesen
Copy link
Owner Author

Without a way to migrate with the braking changes?

I'm not calling this a breaking change as it's still supposed by the framework it will just no longer be the recommended solution for this application so it's been removed as the default.

@alexjustesen
Copy link
Owner Author

Also to note, this impacts new installs not existing.

@AleksCee
Copy link

Also to note, this impacts new installs not existing.

Ok this is a info I can't figure out from the Changnotes.
Then it's no braking change. 👍🏻
Anyway it's can be a good idea to migrate the data, are you plan a support feature to do that?

Thx, Alex

@alexjustesen
Copy link
Owner Author

@AleksCee once I'm through these Docker issues import/export/backup is the next priority.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🧹 chore Nothing fancy, just needs to get done
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants