Skip to content

rpc: allow to getcontractstate by id or name#1426

Merged
roman-khimov merged 5 commits intomasterfrom
rpc/getcontractstate_by_id_or_name
Nov 5, 2020
Merged

rpc: allow to getcontractstate by id or name#1426
roman-khimov merged 5 commits intomasterfrom
rpc/getcontractstate_by_id_or_name

Conversation

@AnnaShaleva
Copy link
Copy Markdown
Member

@AnnaShaleva AnnaShaleva commented Sep 25, 2020

close #1423

@codecov
Copy link
Copy Markdown

codecov bot commented Sep 25, 2020

Codecov Report

Merging #1426 into master will increase coverage by 0.08%.
The diff coverage is 60.86%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1426      +/-   ##
==========================================
+ Coverage   77.46%   77.55%   +0.08%     
==========================================
  Files         229      229              
  Lines       17801    17869      +68     
==========================================
+ Hits        13790    13858      +68     
+ Misses       3056     3046      -10     
- Partials      955      965      +10     
Impacted Files Coverage Δ
cli/wallet/multisig.go 60.71% <0.00%> (ø)
pkg/rpc/client/nep5.go 48.67% <ø> (ø)
cli/wallet/nep5.go 73.13% <23.07%> (+6.58%) ⬆️
cli/wallet/validator.go 71.87% <40.00%> (-1.90%) ⬇️
cli/wallet/wallet.go 62.86% <42.85%> (-0.24%) ⬇️
pkg/rpc/client/client.go 74.02% <60.00%> (-2.10%) ⬇️
pkg/rpc/client/rpc.go 85.93% <61.53%> (-1.06%) ⬇️
pkg/rpc/server/server.go 78.03% <80.00%> (-0.07%) ⬇️
pkg/core/blockchain.go 73.49% <100.00%> (+0.12%) ⬆️
pkg/core/native/contract.go 96.61% <100.00%> (+0.24%) ⬆️
... and 2 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7121686...fc018af. Read the comment docs.

@AnnaShaleva AnnaShaleva marked this pull request as draft September 25, 2020 17:00
@AnnaShaleva AnnaShaleva force-pushed the rpc/getcontractstate_by_id_or_name branch 2 times, most recently from 19178cc to 10304ef Compare September 28, 2020 09:27
@AnnaShaleva AnnaShaleva force-pushed the rpc/getcontractstate_by_id_or_name branch 4 times, most recently from 3235fb5 to 7fefa9a Compare November 2, 2020 11:28
@AnnaShaleva AnnaShaleva marked this pull request as ready for review November 2, 2020 11:28
Copy link
Copy Markdown
Member

@roman-khimov roman-khimov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We also need to document our protocol extensions.

@AnnaShaleva AnnaShaleva force-pushed the rpc/getcontractstate_by_id_or_name branch 6 times, most recently from ad6fe5e to 3ef9383 Compare November 3, 2020 11:47
@AnnaShaleva AnnaShaleva force-pushed the rpc/getcontractstate_by_id_or_name branch from 3ef9383 to c07c54b Compare November 3, 2020 15:06
Copy link
Copy Markdown
Member

@roman-khimov roman-khimov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, it's all fine except for verify which should be moved out of this PR (and then developed in sync with C# codebase).

@AnnaShaleva AnnaShaleva force-pushed the rpc/getcontractstate_by_id_or_name branch from c07c54b to fc018af Compare November 5, 2020 07:38
@roman-khimov roman-khimov merged commit 679846c into master Nov 5, 2020
@roman-khimov roman-khimov deleted the rpc/getcontractstate_by_id_or_name branch November 5, 2020 10:17
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.

Add support for querying contracts by ID and name

2 participants