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

LAM-1169 feat: cache configs to disk #1190

Open
wants to merge 27 commits into
base: dev
Choose a base branch
from

Conversation

siiky
Copy link
Contributor

@siiky siiky commented Sep 20, 2024

No description provided.

@siiky siiky force-pushed the feat/lam-1169/cache-configs-to-disk branch 6 times, most recently from b60ac56 to fa6c436 Compare September 26, 2024 15:23
@siiky siiky force-pushed the feat/lam-1169/cache-configs-to-disk branch from 75104d2 to 54ce796 Compare October 1, 2024 14:41
@siiky siiky changed the title LAM-1169 cache configs to disk LAM-1169 feat: cache configs to disk Oct 2, 2024
@siiky siiky force-pushed the feat/lam-1169/cache-configs-to-disk branch from 79eb3ac to bce5003 Compare October 11, 2024 17:17
@siiky siiky marked this pull request as ready for review October 11, 2024 17:20
lib/variable-interval.js Outdated Show resolved Hide resolved
lib/sqlite.js Outdated Show resolved Hide resolved
lib/sqlite.js Outdated Show resolved Hide resolved
lib/db.js Outdated Show resolved Hide resolved
lib/sqlite.js Outdated Show resolved Hide resolved
lib/sqlite.js Outdated Show resolved Hide resolved
lib/sqlite.js Outdated Show resolved Hide resolved
Copy link
Collaborator

@RafaelTaranto RafaelTaranto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Another more general thing. This would benefit from having a separate folder under lib, either /db, /settings, /config or something in this vein. We can then have a pretty closed module.

Would be good to add testing for this as well, you can use vitest, maybe will have to search on how to use vitest without vite.

@siiky siiky force-pushed the feat/lam-1169/cache-configs-to-disk branch 8 times, most recently from 1497f45 to 309bf26 Compare October 17, 2024 11:17
@siiky siiky force-pushed the feat/lam-1169/cache-configs-to-disk branch 4 times, most recently from 07edf78 to 987eb15 Compare October 17, 2024 11:23
@siiky siiky force-pushed the feat/lam-1169/cache-configs-to-disk branch 4 times, most recently from 32650cc to 6bcdd55 Compare October 17, 2024 14:45
@siiky siiky force-pushed the feat/lam-1169/cache-configs-to-disk branch from 6bcdd55 to e4ce4c0 Compare October 17, 2024 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants