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

(Refactor) : Updated few dependencies related to : React, React-Redux #202

Conversation

Abhijay007
Copy link
Contributor

Summary of Changes

This pull request aims to update several dependencies to their latest versions to ensure compatibility, security, and leverage new features and enhancements. The following dependencies have been updated:

Related Issue

Resloves #171

Testing:

  • Tested updated dependencies locally to ensure functionality remains intact.

    Used commands:

    • yarn test - all tests passed
    • yarn dev - no UI fails detected
    • yarn build - no build fails detected
    • yarn dev - no new console errors detected for any dependencies

Checklist

  • My code is well-documented, and I've updated relevant documentation.
  • I have tested these changes on my local environment.
  • I have reviewed and proofread my code and the changes.
  • The branch is up-to-date with the base branch.

Additional Context

Reviewer(s)

@lrasmus

Signed-off-by: Abhijay007 <Abhijay007j@gmail.com>
@Abhijay007 Abhijay007 marked this pull request as draft April 12, 2024 10:21
Copy link
Contributor

@lrasmus lrasmus left a comment

Choose a reason for hiding this comment

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

@Abhijay007 thank you for the continued updates! I tried to test this, and have continued to get the following error. I removed my node_modules folder and did a full install just to be sure, and am able to reproduce the error each time. I'm assuming you didn't see this - I may need a little time to investigate and resolve whatever is going on.

ERROR in ./node_modules/react-redux/dist/react-redux.mjs 1103:18-45
export 'useSyncExternalStore' (imported as 'React2') was not found in 'react' (possible exports: Children, Component, Fragment, Profiler, PureComponent, StrictMode, Suspense, __SECRET_INTERNALS_DO_NOT_USE_OR_YOU_WILL_BE_FIRED, cloneElement, createContext, createElement, createFactory, createRef, forwardRef, isValidElement, lazy, memo, useCallback, useContext, useDebugValue, useEffect, useImperativeHandle, useLayoutEffect, useMemo, useReducer, useRef, useState, version)
 @ dll renderer renderer[49]

webpack 5.91.0 compiled with 1 error and 3 warnings in 6547 ms

@Abhijay007
Copy link
Contributor Author

@Abhijay007 thank you for the continued updates! I tried to test this, and have continued to get the following error. I removed my node_modules folder and did a full install just to be sure, and am able to reproduce the error each time. I'm assuming you didn't see this - I may need a little time to investigate and resolve whatever is going on.

ERROR in ./node_modules/react-redux/dist/react-redux.mjs 1103:18-45
export 'useSyncExternalStore' (imported as 'React2') was not found in 'react' (possible exports: Children, Component, Fragment, Profiler, PureComponent, StrictMode, Suspense, __SECRET_INTERNALS_DO_NOT_USE_OR_YOU_WILL_BE_FIRED, cloneElement, createContext, createElement, createFactory, createRef, forwardRef, isValidElement, lazy, memo, useCallback, useContext, useDebugValue, useEffect, useImperativeHandle, useLayoutEffect, useMemo, useReducer, useRef, useState, version)
 @ dll renderer renderer[49]

webpack 5.91.0 compiled with 1 error and 3 warnings in 6547 ms

@lrasmus I am aware about the error, I will soon resolve it, I marked this PR as draft will request the review soon

Signed-off-by: Abhijay007 <Abhijay007j@gmail.com>
Signed-off-by: Abhijay007 <Abhijay007j@gmail.com>
@Abhijay007 Abhijay007 marked this pull request as ready for review April 18, 2024 16:31
Copy link
Contributor

@lrasmus lrasmus left a comment

Choose a reason for hiding this comment

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

Sorry for the delayed review, changes all look good. Thank you!

@lrasmus lrasmus merged commit a5c7627 into StatTag:master Apr 25, 2024
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