Fixed the unlock profile issue

[abhishek-01k]

Pull Request Template

Fixed the unlock profile issue when in remember me mode

Ticket Number

• Ticket Number

Description

• Problem/Feature:

Type of Change

• 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

• Followed frontend guidelines as per the Guidelines Notion Doc

Build & Testing

• 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]

• There is a missing closing curly brace '}' after the line 'const createdUser = await PushAPI.user.get({'.
• There is a missing closing parenthesis ')' at the end of the line 'const pvtkey = await PushAPI.chat.decryptPGPKey({'.
• It seems like there is a logic issue where 'return (<AppContext.Provider...' is written inside the createUserIfNecessary function, which is not a valid return statement for a Promise function.
• It is recommended to remove the commented-out code to maintain clarity and cleanliness in the codebase.

// Update the code as per the corrections mentioned above
...

      const signer = await provider.getSigner();
      await PushAPI.user.create({
        account: account,
        env: appConfig.appEnv,
        signer: signer,
        progressHook: onboardingProgressReformatter,
      });
      const createdUser = await PushAPI.user.get({
        // Corrected missing closing curly brace
      });
      const pvtkey = await PushAPI.chat.decryptPGPKey({
        encryptedPGPPrivateKey: createdUser.encryptedPrivateKey,
        toUpgrade: true,
      }); // Corrected missing closing parenthesis
      const createdConnectedUser = { ...createdUser, privateKey: pvtkey };
      setConnectedUser(createdConnectedUser);
      setPgpPvtKey(pvtkey);
      return createdConnectedUser;
    } catch (e) {
      console.error(e);
      // Handle the error appropriately
      throw e;
    }
  };

  return (
    <AppContext.Provider
      value={{
        web3NameList,
        setWeb3NameList,
        MetamaskPushSnapModalComponent,
        showMetamaskPushSnap,
        setSnapState,
        initializePushSDK,
        // Other values to be provided
      }}
    >
      {props.children}
    </AppContext.Provider>
  );

...

All looks good.

[github-actions]

PR Preview Action v1.4.7
Preview removed because the pull request was closed.
2024-10-16 09:16 UTC

[github-actions]

1. In the shouldCreateNewPushUser function, there is a missing closing parenthesis for the return statement. It should be added before the closing curly brace:

return !(
  userPushSDKInstance?.account === checksumAddr &&
  userPushSDKInstance?.env === appConfig.appEnv &&
  userPushSDKInstance?.signer === signer &&
  !userPushSDKInstance.readmode()
);

2. In the initializePushSdkGuestMode function, there is a missing closing curly brace at the end of the function. It should be added:

dispatch(setUserPushSDKInstance(userInstance));
};

3. In the initializePushSdkReadMode function, the initializePushSDK(wallet); call is missing a return statement:

return initializePushSDK(wallet);

4. In the initializePushSdkReadMode function, the call to PushAPI.initialize is missing the second parameter object. It should be:

const userInstance = await PushAPI.initialize({
  decryptedPGPPrivateKey: null,
  account: account,
});

5. In the initializePushSdkReadMode function, there is a missing closing curly brace before the await setupStream(userInstance); statement. It should be added:

});

6. In the initializePushSDK function, there is a missing closing curly brace at the end of the function. It should be added:

});

After addressing these issues, the code should be verified, and if everything else is in order, you can write 'All looks good.'

[github-actions]

• There is a missing closing brace '}' after the line const createdUser = await PushAPI.user.get({.
• In the line const pvtkey = await PushAPI.chat.decryptPGPKey({, there is a missing closing brace '}' for the PushAPI.chat.decryptPGPKey function call.
• The line const createdConnectedUser = { ...createdUser, privateKey: pvtkey }; should be outside of the try block to ensure it is reachable.
• In the catch block, the return statement return (<AppContext.Provider ... seems incorrect. It should be handling the error case properly instead of returning JSX. Maybe log the error and handle it accordingly.

All mentioned corrections should be reviewed and implemented before merging this code.