Skip to content

Upgrade rxjs from 5.x to 7.x #167

Description

@2ndtlmining

Carried over from CODE_IMPROVEMENT_PLAN.md (Phase 5C), which is being retired as a standalone doc now that everything else in it has shipped.

client/package.json currently pins "rxjs": "^5.6.0-forward-compat.5" (plus localforage-observable, which depends on it) — old, not tree-shakeable, and long past upstream support. Upgrading to 7.x is a breaking change: every subscription in MainApp.jsx, Home.jsx, and LayoutContext.jsx needs review and likely a rewrite to the pipe-based API.

Risk: High — breaking API change across the app's core data flow.
Suggested prerequisite: land #166 first, to reduce the number of files this touches.

Activity

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions