fix: portfolio wallet deposit modal vault select

edwinzhng commented 2 years ago

Description

Currently, the transaction deposit modal expects the URL path to be /wallet when it sets the allowVaultSelect parameter true, this doesn't happen right now since the new page URL path is /portfolio
I updated the code to pass in props to the modal instead of reading the path, so that this is more explicit and shouldn't happen again if we ever change the page URL path later down the road

Related Issue

Fixes #626

Motivation and Context

Now users can select a vault to zap into again instead of just the default

How Has This Been Tested?

Manually tested on portfolio wallet page with different tokens. Also tested the vault page and confirmed that functionality is still the same there (can change token but cannot change vault to deposit into).

Screenshots (if appropriate):

Screen Shot 2022-05-03 at 5 11 08 PM

Screen Shot 2022-05-03 at 5 13 19 PM

aws-amplify-us-east-1[bot] commented 2 years ago

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-628.d27dgpz01hmbvx.amplifyapp.com