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

new wallet changes #1200

Merged
merged 3 commits into from
Jul 28, 2023
Merged

new wallet changes #1200

merged 3 commits into from
Jul 28, 2023

Conversation

corlard3y
Copy link
Collaborator

No description provided.

@corlard3y corlard3y requested a review from 0xNilesh July 18, 2023 20:36
@github-actions
Copy link

All looks good.

@corlard3y corlard3y requested review from deekshithepns and mishramonalisha76 and removed request for deekshithepns July 19, 2023 11:44
@github-actions
Copy link

File: src/AppLogin.tsx

  • Line 19: The prop "fontSize" should be "32px" instead of 32px.
  • Line 79: There is a missing closing parenthesis ")" for the try-catch block.
  • Line 83: There is a missing closing curly brace "}" for the catch block.
  • Line 88: There is a missing closing parenthesis ")" for the catch block.
  • Line 92: There is a missing closing curly brace "}" for the catch block.
  • Line 98: There is a missing closing parenthesis ")" for the catch block.
  • Line 103: There is a missing closing curly brace "}" for the catch block.

File: src/helpers/ChainHelper.ts

  • Line 40: The function handleChangeAllowedNetwork expects three parameters, but only two are provided. The third parameter "connector" is missing.

File: src/hooks/useEagerConnect.ts

  • Line 32: There is a missing closing parenthesis ")" after the getAddChainParameters function call.
  • Line 37: There is a missing closing curly brace "}" for the switch statement.
  • Line 43: There is a missing closing parenthesis ")" for the try-catch block.

All looks good.

@0xNilesh 0xNilesh merged commit f179b05 into main Jul 28, 2023
1 check passed
@0xNilesh 0xNilesh deleted the newWalletChanges branch July 28, 2023 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done and Merged
Development

Successfully merging this pull request may close these issues.

3 participants