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

RC1 to RC2 Comparison #7

Open
wants to merge 29 commits into
base: rc1-untouched
Choose a base branch
from
Open

RC1 to RC2 Comparison #7

wants to merge 29 commits into from

Conversation

spencercorwin
Copy link

No description provided.

shargon and others added 29 commits March 18, 2021 09:37
* UT Codding Style

* dotnet format
* Role event

* Add abi and blockNumber

* Rename

* Update src/neo/SmartContract/Native/RoleManagement.cs

Co-authored-by: Erik Zhang <erik@neo.org>

* Update requireFlags

Co-authored-by: Erik Zhang <erik@neo.org>
* add network magic to ContractParametersContext

* renew note

* change magic to network in json

* fix

* Change

* Change to network

* Rename

Co-authored-by: Shargon <shargon@gmail.com>
Co-authored-by: Qiao Jin <jinqiao0423@gmail.com>
* Prevent negative GAS

* Create handler for System.Runtime.BurnGas

Co-authored-by: Erik Zhang <erik@neo.org>
…n inside CalculateNetworkFee method (neo-project#2414)

* Update calculate fee

* Fix

* Update src/neo/Wallets/Wallet.cs

Co-authored-by: AnnaShaleva <shaleva.ann@gmail.com>

* Fix size

* Update src/neo/Wallets/Wallet.cs

Co-authored-by: AnnaShaleva <shaleva.ann@gmail.com>

* Remove foreach

* Or 0

Co-authored-by: AnnaShaleva <shaleva.ann@gmail.com>
Co-authored-by: Erik Zhang <erik@neo.org>
* Add more functions to StdLib

* Use attributes (neo-project#2424)

* Apply suggestions from code review

Co-authored-by: Shargon <shargon@gmail.com>

* fix

* Add UT

* Update prices

* removeEmptyEntries

Co-authored-by: Shargon <shargon@gmail.com>
* Check if the PK it's a valid one

* Optimization

Co-authored-by: Erik Zhang <erik@neo.org>
* Ensure ExtensiblePayload

* Remove check sender

Co-authored-by: Erik Zhang <erik@neo.org>
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.

8 participants