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 farming modal.
Notice
Pull Request Type
Issue (if applicable)
Risk
Reconciliation could be wrong and rug the FOX-ETH farming 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 -FoxLP
can also be enabled to test for FOX LP regressionsScreenshots (if applicable)