Closed gomesalexandre closed 1 year ago
Current dependencies on/for this PR:
This comment was auto-generated by Graphite.
Moving to draft pending stack merge issues resolution
Current dependencies on/for this PR:
This comment was auto-generated by Graphite.
Current dependencies on/for this PR:
This comment was auto-generated by Graphite.
Current dependencies on/for this PR:
This comment was auto-generated by Graphite.
Current dependencies on/for this PR:
This comment was auto-generated by Graphite.
Current dependencies on/for this PR:
This comment was auto-generated by Graphite.
Current dependencies on/for this PR:
This comment was auto-generated by Graphite.
Current dependencies on/for this PR:
This comment was auto-generated by Graphite.
Current dependencies on/for this PR:
This comment was auto-generated by Graphite.
Current dependencies on/for this PR:
This comment was auto-generated by Graphite.
Current dependencies on/for this PR:
This comment was auto-generated by Graphite.
Current dependencies on/for this PR:
This comment was auto-generated by Graphite.
Description
This adds initial support for account selection in the FOX-ETH deposit/withdraw steps of the LP modal.
Note: Syncing won't work in the current state as this requires
<AccountDropdown />
to be optionally controlled, which currently isn't the case.Notice
Pull Request Type
Issue (if applicable)
Risk
Reconciliation could be wrong and rug the FOX-ETH LP feature, test accordingly.
Testing
MultiAccount
flag onFoxFarming
andFoxLP
flags on as Fox Farming and LP opportunities will never be displayed otherwise)FoxFarming
andFoxLP
flags on as Fox Farming and LP opportunities will never be displayed otherwise) - test the full flow of the modal, including deposit/withdrawEngineering
setAccountId
is calledOperations
FoxFarming
andMultiAccount
flags on to test this -FoxFarming
can also be enabled to test for FOX Farming regressionsScreenshots (if applicable)