-
Notifications
You must be signed in to change notification settings - Fork 159
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
What is the reason of exposing only accountId, but not entire ConnectedWalletAccount? #429
Comments
Curious about this too. Is the intended pattern that |
Exposing |
but near-api-js are already part of wallet selector dependencies.
|
Injected scripts do not have direct access to these dependencies. |
@karpovdmitryod Not all wallets use the If you have any other questions feel free to reopen this issue. |
No updates on this.....? Very bad design to not allow for full ConnectedWalletAccount object...... |
During the migration to WalletSelector, we faced the fact that you are not exposing ConnectedWalletAccount object.
Due to this, many functionalities available before switching to WalletSelected are no longer accessible.
e.g - you just recently added the possibility to get a user signature
another example - we cannot create a new Contract instance, and we are now rewriting everything to RPC calls.
What was the reason for not exposing this object to the end users?
The text was updated successfully, but these errors were encountered: