Sign typed data using account object instead of account address for the tradecoin api

[ericli]

Description

Signing with the account instead of the account address in the tradecoin api

Motivation and Context

I'm facing an issue where I cannot use my local signed account. I noticed that we sign using the account address instead of the authed account.

Does this change the ABI/API?

• This changes the ABI/API

What tests did you add/modify to account for these changes

Types of changes

• [x ] Bug fix (non-breaking change which fixes an issue)
• New module / feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

• [x ] My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I added a changeset to account for this change

Reviewer Checklist:

• My review includes a symposis of the changes and potential issues
• The code style is enforced
• There are no risky / concerning changes / additions to the PR

[iainnash]

Good catch — taking a look on Tuesday

[J3698]

big fan of this one