Skip to content

refactor(renderer)!: remove legacy context storage migration - #3515

Open
setchy wants to merge 1 commit into
mainfrom
refactor/remove-legacy-storage-migration
Open

setchy wants to merge 1 commit into
mainfrom
refactor/remove-legacy-storage-migration

Conversation

@setchy

@setchy setchy commented Oct 25, 2026

Copy link
Copy Markdown
Member

Summary

Removes migrateLegacyStoreToZustand(), the one-shot migration from the v6 unified gitify-storage localStorage key to the split Zustand stores introduced in v7.0.0.

The TODO on the function asked for removal in a future major release, and this lands ahead of the next major, which no longer supports upgrading directly from v6.

Changes

  • Delete src/renderer/utils/core/storage.ts and storage.test.ts
  • Drop the import and top-level call in src/renderer/App.tsx
  • Remove Constants.STORAGE.LEGACY (gitify-storage) from src/renderer/constants.ts

Behaviour change

Users upgrading v6 → v8 directly will no longer have their legacy accounts and settings migrated: they will land on the login screen with default settings, and the old gitify-storage key will remain in localStorage. Users who have run any 7.x release are unaffected, since the migration already consumed and removed that key.

Upgrading via 7.x is the supported path. Worth calling out in the v8 release notes.

Validation

  • pnpm check — clean (lint, format, type-check)
  • pnpm test — 203 files / 1967 tests passing

The migration from the v6 unified `gitify-storage` key to the split Zustand stores has shipped since v7.0.0. Removing it ahead of the next major release, which no longer supports upgrading directly from v6.
@setchy
setchy requested a review from afonsojramos as a code owner October 25, 2026 15:51
@github-actions github-actions Bot added the refactor Refactoring of existing feature label Oct 25, 2026
@sonarqubecloud

Copy link
Copy Markdown

@setchy setchy changed the title refactor(renderer): remove legacy Context storage migration refactor(renderer)!: remove legacy Context storage migration Oct 25, 2026
@setchy setchy changed the title refactor(renderer)!: remove legacy Context storage migration refactor(renderer)!: remove legacy context storage migration Oct 25, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor Refactoring of existing feature

Development

Successfully merging this pull request may close these issues.

1 participant