search

JM-Lemmi / ical-relay

Relay ical urls and edit them on the fly with different modules.
GNU Affero General Public License v3.0
9 stars 2 forks source link

Remove Embedded DB, add Ephemeral Mode #201

Closed JM-Lemmi closed 9 months ago

JM-Lemmi commented 1 year ago

This removes the special embedded DB, since it only starts a standard postgress service in the background.

Instead it introduces an official ephemeral mode, removes the config write ability.

robske110 commented 1 year ago

I don't know if we should remove the embedded db, since it doesn't really relate to the ephemeral mode (it is not ephemeral). It is still useful for testing and could be especially useful if we ever introduce unit tests.

JM-Lemmi commented 1 year ago

I don't want the embedded DB, since its not actually embedded. Its not a database actually in the binary included, but needs internet access on first launch and downloads another binary. That's not expected and imo afe behaviour

I would be in favor of an embedded db that is included in the binary and accesses the database file directly, like i proposed earlier with the sqlite approach.

JM-Lemmi commented 1 year ago

We can also write docker-compose files for development that already include the postgres server