Skip to content

first draft of generic masterchef fantom - #3

Open
flashfish0x wants to merge 7 commits into
dudesahn:mainfrom
flashfish0x:ripae
Open

first draft of generic masterchef fantom#3
flashfish0x wants to merge 7 commits into
dudesahn:mainfrom
flashfish0x:ripae

Conversation

@flashfish0x

Copy link
Copy Markdown

No description provided.

@fp-crypto fp-crypto left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Had one question, otherwise LGTM

address[] memory emissionTokenPath = new address[](2);
emissionTokenPath[0] = address(emissionToken);
emissionTokenPath[1] = address(usdc);
emissionTokenPath[1] = address(poolTwoSecondToken);

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Might this ever need to be swapped via weth/wftm? In the case where address(want) == address(poolTwoSecondToken)?

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

agree, one of the vaults is WFTM and the pool 2 is with WFTM, so we shouldn't need the second sell for it

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nevermind, this should be handled just fine as-is

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

telegram-cloud-photo-size-4-6037309243520038344-x
if want == secondtoken then we dont do the second

@dudesahn

dudesahn commented Feb 1, 2022

Copy link
Copy Markdown
Owner

Would also be good to add an option for

emission -> something -> WFTM -> want

as this is a template

@flashfish0x

Copy link
Copy Markdown
Author

Would also be good to add an option for

emission -> something -> WFTM -> want

as this is a template

3b88bc8

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants