Skip to content

Conversation

SAHU-01
Copy link
Contributor

@SAHU-01 SAHU-01 commented Aug 31, 2025

Following the REDUX-BUNDLER-MIGRATION-GUIDE.md this PR migrates local-storage.js bundler to fix part of #2349

Changes:

  • Created: src/contexts/tours-context.tsx - React context managing tour state and tooltip preferences

  • Updated: src/App.js - Added ToursProvider to app hierarchy

  • Migrated components from redux-bundler HOCs to React hooks:
    ✅ src/files/FilesPage.js - Removed withTour HOC, uses useTours() hook, creates local handleJoyrideCallback
    ✅ src/peers/PeersPage.js - Removed withTour HOC, uses useTours() hook, creates local handleJoyrideCallback
    ✅ src/welcome/WelcomePage.js - Removed withTour HOC, uses useTours() hook, creates local handleJoyrideCallback
    ✅ src/settings/SettingsPage.js - Removed withTour HOC, wrapper component with useTours() hook
    ✅ src/status/StatusPage.js - Removed withTour HOC, uses useTours() hook, creates local handleJoyrideCallback
    ✅ src/App.js - Added wrapper component for tooltip functionality

  • Updated: src/components/tour/withTour.js - Now uses context internally (kept for potential future use)

  • Removed:
    ✅ src/bundles/tours.js from bundle composition
    ✅ withTour HOC dependency from all migrated components

Pattern:

  • Replaced selectToursEnabled, selectShowTooltip selectors with useTours() hook
  • Replaced doEnableTours, doDisableTours, doDisableTooltip actions with context methods
  • Removed withTour(Component) wrapper pattern in favor of direct hook usage
  • Components create their own handleJoyrideCallback using disableTours from context

Reviewer:
@SgtPooki

@SAHU-01 SAHU-01 requested a review from a team as a code owner August 31, 2025 05:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant