Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implemented: Added the support for using productStore selector from dxp-component #722

Merged
merged 10 commits into from
Dec 18, 2024

Conversation

R-Sourabh
Copy link
Contributor

Related Issues

hotwax/dxp-components#193

Short Description and Why It's Useful

  • Added support for using the productStore selector on the settings page from the dxp-component.
  • Replaced the app-level productStore state and updated the logic to use the dxp-component for productStore state management.
  • Removed code related to storing/fetching productStores and currentProductStore in the application.
  • Utilized getters from the user store in dxp-components across the entire application.

Contribution and Currently Important Rules Acceptance

@R-Sourabh R-Sourabh changed the title Improved: Added the support for using productStore selector from dxp-component [WIP] Improved: Added the support for using productStore selector from dxp-component Sep 2, 2024
@R-Sourabh R-Sourabh changed the title [WIP] Improved: Added the support for using productStore selector from dxp-component Implemented: Added the support for using productStore selector from dxp-component Dec 17, 2024
Copy link
Contributor

@ymaheshwari1 ymaheshwari1 left a comment

Choose a reason for hiding this comment

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

Revert changes of package lock file as we have not updated package.json file

src/store/modules/util/actions.ts Outdated Show resolved Hide resolved
@ymaheshwari1 ymaheshwari1 merged commit 99f2f8c into hotwax:main Dec 18, 2024
2 checks passed
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.

2 participants