Fix/connectors account

[bluecco]

Issue / feature description

• account method for connectors are using starknetjs WalletAccount

this is causing issues for webwallet and argent in app browsers since a request is performed in the constructor

walletProvider
      .request({
        type: 'wallet_requestAccounts',
        params: {
          silent_mode: false,
        },
      })
      .then((res) => {
        this.address = res[0].toLowerCase();
      });

This call will display an additional popup for webwallet and in app browser connection due to the wallet request

Changes

• update account method to use an Account instead of WalletAccount since the return type is an AccountInterface anyway
• connect callback doesn't perform connector.connect() if the resultType is connector

Checklist

• Rebased to the last commit of the target branch (or merged)
• Code self-reviewed
• Code self-tested
• Tests updated (if needed)
• All tests are passing locally

Please keep your pull request as small as possible. If you need to make multiple changes, please create separate pull requests for each change. Create a draft pull request if you need early feedback. This will mark the pull request as a work in progress and prevent it from being reviewed or merged until you are ready.

Please move drafts to the ready for review (regular PR) when you are ready to start the review process and other developers will pick it up from there.

[Cussone]

Looking good to me 👍🏼