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
fix-transferRole-check-on-esdtTransfer #4014
Conversation
Codecov Report
@@ Coverage Diff @@
## rc/2022-may #4014 +/- ##
===============================================
- Coverage 74.79% 74.78% -0.01%
===============================================
Files 609 609
Lines 80935 80944 +9
===============================================
Hits 60536 60536
- Misses 15764 15771 +7
- Partials 4635 4637 +2
Continue to review full report at Codecov.
|
ESDTTransferRoleEnableEpoch: enableEpochs.ESDTTransferRoleEnableEpoch, | ||
GlobalMintBurnDisableEpoch: enableEpochs.GlobalMintBurnDisableEpoch, | ||
ESDTTransferMetaEnableEpoch: enableEpochs.BuiltInFunctionOnMetaEnableEpoch, | ||
OptimizeNFTStoreEnableEpoch: enableEpochs.OptimizeNFTStoreEnableEpoch, |
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.
this was a forgotten flag. Does the code is still backwards compatible now?
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.
having a second thought, let's keep it this way as this is the correct way. Will deal with the import-db process afterwards
go.mod
Outdated
@@ -12,7 +12,7 @@ require ( | |||
github.com/ElrondNetwork/elrond-go-core v1.1.15 | |||
github.com/ElrondNetwork/elrond-go-crypto v1.0.1 | |||
github.com/ElrondNetwork/elrond-go-logger v1.0.5 | |||
github.com/ElrondNetwork/elrond-vm-common v1.2.12 | |||
github.com/ElrondNetwork/elrond-vm-common v1.2.14-0.20220421185714-edf68ab67a9e |
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.
proper release
123951d
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.
@@ Log scanner @@
fix-transferRole-check-on-transfer
================================================================================
- Known Warnings 450
- New Warnings 342
- Known Errors 0
- New Errors 0
- Panics 0
================================================================================
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.
Looks good 👍
We will wait until the feat/release-2022-April
gets merged and then re-target this in rc/2022-may
Requested changes only to prevent accidental merges
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.
System tests passed.
integrated new flag