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.

Sign up for GitHub

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

Jump to bottom

Impliments debug_traceTransaction #1573

Open
wants to merge 42 commits into
base: master
Choose a base branch
from
Open

Impliments debug_traceTransaction #1573

wants to merge 42 commits into from

Conversation

Sriep
Copy link
Contributor

@Sriep Sriep commented Nov 7, 2019 •
edited
Loading

  • I added unit tests for any code that added
  • I updated the CHANGELOG.md
  • All IP is original and not copied from another source
  • I assign all copyright to Loom Network for the code in the pull request

issue #1482
Implements debug_traceTransaction as outlined in Management APIs
Implments debug_storageRangeAt

Needs two other PRs

@Sriep Sriep requested a review from enlight November 7, 2019 17:31
enlight
enlight requested changes Nov 8, 2019
View reviewed changes
e2e/tests/truffle/test/DebugFunctions.js Outdated Show resolved Hide resolved
rpc/query_server_test.go Outdated Show resolved Hide resolved
store/versioned_cachingstore.go Show resolved Hide resolved
store/versioned_cachingstore.go Outdated Show resolved Hide resolved
evm/loomevm.go Outdated Show resolved Hide resolved
@Sriep Sriep mentioned this pull request Nov 8, 2019
Open
enlight
enlight requested changes Nov 11, 2019
View reviewed changes
e2e/tests/truffle/test/DebugFunctions.js Outdated Show resolved Hide resolved
e2e/tests/truffle/test/DebugFunctions.js Outdated Show resolved Hide resolved
store/versioned_cachingstore.go Show resolved Hide resolved
rpc/debug/trace.go Outdated Show resolved Hide resolved
tx_handler/factory/handler_factory.go Show resolved Hide resolved
evm/loomevm.go Outdated Show resolved Hide resolved
evm/loomevm.go Outdated Show resolved Hide resolved
enlight
enlight requested changes Nov 15, 2019
View reviewed changes
config/config.go Outdated Show resolved Hide resolved
store/versioned_cachingstore.go Outdated Show resolved Hide resolved
evm/evm.go Show resolved Hide resolved
evm/loomevm.go Outdated Show resolved Hide resolved
pathornteng
pathornteng suggested changes Nov 19, 2019
View reviewed changes
tx_handler/factory/handler_factory.go Outdated Show resolved Hide resolved
store/versioned_cachingstore.go Outdated Show resolved Hide resolved
app.go Outdated Show resolved Hide resolved
store/splitstore.go Outdated Show resolved Hide resolved
store/multi_writer_app_store.go Show resolved Hide resolved
enlight
enlight requested changes Nov 21, 2019
View reviewed changes
app.go Show resolved Hide resolved
config/config.go Outdated Show resolved Hide resolved
store/splitstore.go Outdated Show resolved Hide resolved
store/splitstore.go Show resolved Hide resolved
rpc/debug/trace.go Show resolved Hide resolved
enlight
enlight requested changes Nov 26, 2019
View reviewed changes
store/splitstore.go Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pathornteng pathornteng pathornteng requested changes

@enlight enlight enlight requested changes

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Sriep @enlight @pathornteng