Skip to content

chore: add localStorage backed store (@fehmer)#7540

Merged
Miodec merged 7 commits intomasterfrom
feature/ls-store
Feb 26, 2026
Merged

chore: add localStorage backed store (@fehmer)#7540
Miodec merged 7 commits intomasterfrom
feature/ls-store

Conversation

@fehmer
Copy link
Member

@fehmer fehmer commented Feb 26, 2026

No description provided.

@monkeytypegeorge monkeytypegeorge added the frontend User interface or web stuff label Feb 26, 2026
@Miodec Miodec requested a review from Copilot February 26, 2026 18:55
@github-actions github-actions bot added the waiting for review Pull requests that require a review before continuing label Feb 26, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new SolidJS hook that exposes a createStore-backed state persisted to localStorage via the existing LocalStorageWithSchema utility, with optional cross-tab sync via storage events.

Changes:

  • Introduces useLocalStorageStore hook returning [store, setStore] persisted to localStorage
  • Adds optional cross-tab/window sync using window.storage events
  • Adds hook-local options type and JSDoc/example

@github-actions
Copy link
Contributor

Continuous integration check(s) failed. Please review the failing check's logs and make the necessary changes.

@github-actions github-actions bot added waiting for update Pull requests or issues that require changes/comments before continuing and removed waiting for review Pull requests that require a review before continuing labels Feb 26, 2026
@github-actions github-actions bot removed the waiting for update Pull requests or issues that require changes/comments before continuing label Feb 26, 2026
@Miodec Miodec merged commit 3e700e3 into master Feb 26, 2026
13 checks passed
@Miodec Miodec deleted the feature/ls-store branch February 26, 2026 21:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

frontend User interface or web stuff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants