UserPushSDKInstance returns null randomly on dapp

[corlard3y]

Pull Request Template

1718

Description

• Problem/Feature: UserPushSDKInstance return null randomly on dapp, it is getting reset from the redux state. It is because the UseEffect that resets userSlice from redux state runs everytime, even when account(dependency array) does not change. I added a mounted useRef logic to make it update only when account changes

Type of Change

• [x] Bug fix
• [ ] New feature
• [ ] Code refactor
• [ ] Documentation update
• [ ] Other (please describe):

Checklist

• [ ] Quick PR: Is this a quick PR? Can be approved before finishing a coffee.
  • [ ] Quick PR label added
• [ ] Not Merge Ready: Is this PR dependent on some other PR/tasks and not ready to be merged right now.
  • [ ] DO NOT Merge PR label added

Frontend Guidelines

• [x] Followed frontend guidelines as per the Guidelines Notion Doc

Build & Testing

• [x] No errors in the build terminal
• [ ] Engineer has tested the changes on their local environment
• [ ] Engineer has tested the changes on deploy preview

Screenshots/Video with Explanation

• Before: Explain the previous behavior

• After: What's changed now

Additional Context

Review & Approvals

• [ ] Self-review completed
• [ ] Code review by at least one other engineer
• [ ] Documentation updates if applicable

Notes

[github-actions[bot]]

There are some issues in the code snippet provided:

1. There is a typo in the function setcurrentTime, it should be setCurrentTime with a capital 'C'.
2. The function extendConsole is defined but not being called, consider removing it if it's unnecessary.
3. There is a missing closing brace } after console.enable('warn', false); inside the if (appConfig?.appEnv === 'prod') block.
4. There are unused imports that can be removed to clean up the code.
5. Inside the if (!mounted.current) block, there is a missing closing brace } after the resetState(); call.
6. The useEffect hook is missing its dependency array in the App component, you should pass [account] as the second argument.
7. Inside the toggleDarkMode function, there is a missing closing brace after the document.documentElement.setAttribute('theme', theme);.
8. The SidebarCollapsable variable is misspelled, it should be SidebarCollapsible.

After addressing those issues, the code looks good.

[github-actions[bot]]

PR Preview Action v1.4.7 :---: Preview removed because the pull request was closed. 2024-07-24 09:47 UTC

[github-actions[bot]]

In the App.tsx file:

1. There are some typos, for example, the function name setcurrentTime should be corrected to setCurrentTime.
2. There is a missing curly brace } after the first if condition block before the if (appConfig?.appEnv === 'prod').
3. There is a missing closing curly brace } at the end of the file after the last ... ellipsis.

Other than these issues, the code seems to be sound regarding logic and structure.

In the yarn.lock file: All looks good.