Skip to content

Conversation

@omid2007hope
Copy link

@omid2007hope omid2007hope commented Dec 10, 2025

Overview

This pull request integrates Redux Toolkit into the application, adds a global store,
implements a basket slice with add/remove logic, and wraps the app with the Redux Provider.
It also fixes a broken Husky pre-commit hook and removes a duplicated-key issue in the pnpm-lock.yaml.

Changes Included

  • Added Redux store (configureStore) and connected root reducer.
  • Added basket slice with:
    • addItem
    • removeItem
    • localStorage persistence
  • Wrapped App.jsx with <Provider store={store}>.
  • Fixed .husky/pre-commit to use npm run pre-commit instead of yarn.
  • Cleaned duplicated mapping entry in pnpm-lock.yaml that caused install failures.

Type of Change

  • New feature (non-breaking, adds functionality)
  • Fix (resolves Husky hook error + pnpm lockfile issue)
  • Breaking change (not applicable)

How Has This Been Tested?

  • Manual testing in local environment:
    • App renders correctly with Redux Provider.
    • Slice actions update state and persist to localStorage.
    • pnpm/npm installs run without lockfile errors.
    • Husky pre-commit hook no longer throws “yarn: command not found”.

Additional Notes

These changes merge cleanly and do not modify existing components outside Redux integration.
This prepares the project for scalable state management.

@netlify
Copy link

netlify bot commented Dec 10, 2025

👷 Deploy request for reactplayio pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit c3aaa58

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Hey! contributor, thank you for opening a Pull Request 🎉.

@reactplay/maintainers will review your submission soon and give you helpful feedback. If you're interested in continuing your contributions to open source and want to be a part of a welcoming and fantastic community, we invite you to join our ReactPlay Discord Community.
Show your support by starring ⭐ this repository. Thank you and we appreciate your contribution to open source!
Stale Marking : After 30 days of inactivity this issue/PR will be marked as stale issue/PR and it will be closed and locked in 7 days if no further activity occurs.

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