Skip to content
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

Vesting Extension enabled in mainnet cfg #592

Merged
merged 2 commits into from
Mar 6, 2023
Merged

Conversation

fforbeck
Copy link
Contributor

@fforbeck fforbeck commented Mar 6, 2023

Proposed Changes

  • InternalTokenVestingExtension is used by the ERC20 transfer strategy so we need to enable the extension and factory as part of the mainnet+gorli deployment configuration files.

@fforbeck fforbeck requested a review from adridadou March 6, 2023 13:34
@codecov
Copy link

codecov bot commented Mar 6, 2023

Codecov Report

Merging #592 (29b56af) into master (459544a) will not change coverage.
The diff coverage is n/a.

❗ Current head 29b56af differs from pull request most recent head 93e5a12. Consider uploading reports for the commit 93e5a12 to get more accurate results

@@           Coverage Diff           @@
##           master     #592   +/-   ##
=======================================
  Coverage   90.84%   90.84%           
=======================================
  Files          55       55           
  Lines        2032     2032           
  Branches      462      462           
=======================================
  Hits         1846     1846           
  Misses        186      186           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

adridadou
adridadou previously approved these changes Mar 6, 2023
@fforbeck fforbeck merged commit cd06d7c into master Mar 6, 2023
@fforbeck fforbeck deleted the enable-vesting-ext branch March 6, 2023 14:27
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.

2 participants