-
Notifications
You must be signed in to change notification settings - Fork 0
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
(migration): redo migration for hub v2 #114
Conversation
…can't cherrypick commit
…od in inviteHuman
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm - pf ¯_(ツ)_/¯
# Conflicts: # src/hub/Hub.sol # src/hub/IHub.sol
…int256[] calldata _amounts, IHubV2 _hubV2)
src/hub/Hub.sol
Outdated
* @param _avatars array of avatar addresses to migrate | ||
* @param _amounts array of amounts in inflationary v1 units to migrate | ||
*/ | ||
function migrate(address _owner, address[] calldata _avatars, uint256[] calldata _amounts) external { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oops, seems to miss a modifier
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
requires changes! needs the modifier
fixes #113