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

Storage and edge case fixes #28

Merged
merged 2 commits into from
Aug 13, 2019
Merged

Storage and edge case fixes #28

merged 2 commits into from
Aug 13, 2019

Conversation

austinabell
Copy link
Contributor

  • Fix state iterator ref
  • Fix state log deep copy ref
  • Removed outdated finalization ref
  • Small refactor and cleaned inconsistent or unnecessary code

Will open to review once I do a full sync

@austinabell austinabell marked this pull request as ready for review August 7, 2019 18:50
@austinabell austinabell requested a review from noot as a code owner August 7, 2019 18:50
@soc1c soc1c merged commit 5cf7f91 into development Aug 13, 2019
@soc1c soc1c deleted the austin/storagefixes branch August 13, 2019 11:36
soc1c pushed a commit that referenced this pull request Aug 13, 2019
* Applied storage fixes and updates

* Minor execution refactor
@soc1c soc1c mentioned this pull request Aug 13, 2019
4 tasks
soc1c added a commit that referenced this pull request Aug 13, 2019
* Replace repository reference for accounts/abi/bind (#29)

* docs: update badges (#34)

* docs: add gitter badge

* docs: add circle-ci badge

* Update README.md

* Update README.md

* Update README.md

* docs: update repository path

* Update README.md

* Tx Encoding and decoding fix (#33)

* Added test for decoding old encoding function

* Fixed decoding issue and improved tests to handle all cases including previous client

* Add compatibility with previous version

* Rename receipt storage variable

* Storage and edge case fixes (#28)

* Applied storage fixes and updates

* Minor execution refactor
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants