-
Notifications
You must be signed in to change notification settings - Fork 198
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Frax.mainh #232
Open
Hawthorne001
wants to merge
51
commits into
FraxFinance:FortisFortuna-patch-FraxUnifiedFarm_ERC20_V2
Choose a base branch
from
Hawthorne001:master
base: FortisFortuna-patch-FraxUnifiedFarm_ERC20_V2
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Frax.mainh #232
Hawthorne001
wants to merge
51
commits into
FraxFinance:FortisFortuna-patch-FraxUnifiedFarm_ERC20_V2
from
Hawthorne001:master
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ce#189) * prep for crosschain frxETH * minor fix * compile fix * frxETH ferry stuff * periodic update
* Travis (FraxFinance#172) * farm * compiling * farm 2 * remove unneeded error * segregate changes into V2 for all files altered from original, revert pragma updates on otherwise unchanged files * comment removed * corrected testing errors * move get receiver stake * resolved errors * info * allowance logic * allowanace & approval logic * adds comments & some logic fixes from testing * include comments for allowance, make isApproved external & remove msg.sender from that * convert functions from public to external where feasible to reduce bytecode * file matching * file sync * file sync * file sync * file sync * file sync * enhanced foundry compiler settings * alter a getter * alter a getter * missed fixes from testing * missed fixes from testing * moved getReceiverLockedStake back inside the main if checks for receiver - causes it to fail instead of create a new locked stake Co-authored-by: Travis Moore <[email protected]>
Didn't check whether the receiver address is a contract or not, causing it to fail for instances of transfer to EOA.
Remove unnecessary `{` before the lock transferring checks from line 760.
Change the event emit value to match the name of the value for the event emitted during `_stakeLocked` function. It's emitting the number of stakes the user has instead of the new locked stake index. Added ` - 1` to the value being emitted to get it to match with the name of the parameter for the event.
* Travis (FraxFinance#172) * farm * compiling * farm 2 * remove unneeded error * segregate changes into V2 for all files altered from original, revert pragma updates on otherwise unchanged files * comment removed * corrected testing errors * move get receiver stake * resolved errors * info * allowance logic * allowanace & approval logic * adds comments & some logic fixes from testing * include comments for allowance, make isApproved external & remove msg.sender from that * convert functions from public to external where feasible to reduce bytecode * file matching * file sync * file sync * file sync * file sync * file sync * enhanced foundry compiler settings * alter a getter * alter a getter * missed fixes from testing * missed fixes from testing * moved getReceiverLockedStake back inside the main if checks for receiver - causes it to fail instead of create a new locked stake * bug fixes * remove ending timestamp alteration * changed ILockReceiverV2 back to ILockReceiver since that was original & messes up the selector * overwrote incorrect line --------- Co-authored-by: Travis Moore <[email protected]>
* Travis (FraxFinance#172) * farm * compiling * farm 2 * remove unneeded error * segregate changes into V2 for all files altered from original, revert pragma updates on otherwise unchanged files * comment removed * corrected testing errors * move get receiver stake * resolved errors * info * allowance logic * allowanace & approval logic * adds comments & some logic fixes from testing * include comments for allowance, make isApproved external & remove msg.sender from that * convert functions from public to external where feasible to reduce bytecode * file matching * file sync * file sync * file sync * file sync * file sync * enhanced foundry compiler settings * alter a getter * alter a getter * missed fixes from testing * missed fixes from testing * moved getReceiverLockedStake back inside the main if checks for receiver - causes it to fail instead of create a new locked stake * bug fixes * remove ending timestamp alteration * changed ILockReceiverV2 back to ILockReceiver since that was original & messes up the selector * overwrote incorrect line * add familial middleman gauge type * create familial gauge for redistributing rewards across transferrable & non-transferrable gauges as a signle votable gauge, based on each child gauges total combined weight * remove gauge set up from constructor * change var names & make getters to match with farm required abis * change var names & make getters to match with farm required abis * convert vars to internal to reduce execution gas costs, expose through specified getters * return transfer tests to previous state --------- Co-authored-by: Travis Moore <[email protected]>
zksync stuff and farms
Periodic update. New farms, BAMM, FraxferryV2 WIPs
Fraxoracle and Gnosis Safe Batcher
periodic update
Kyberswap mainly, Gnosis scripts
periodic update
periodic update
routine update
Periodic Update
periodic update
Periodic sync
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.