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

Feature/state_call-api #1326

Merged
merged 6 commits into from
Aug 23, 2022
Merged

Feature/state_call-api #1326

merged 6 commits into from
Aug 23, 2022

Conversation

GaroRobe
Copy link
Contributor

Referenced issues

Fix extrinsics status in polkadot js #1324

Description of the Change

DOT transfer didn't work because RPC call state_call wasn't implemented.
Now it works.

Benefits

Possible Drawbacks

Usage Examples or Tests

Alternate Designs

@GaroRobe GaroRobe linked an issue Aug 22, 2022 that may be closed by this pull request
@GaroRobe GaroRobe self-assigned this Aug 22, 2022
@GaroRobe GaroRobe requested review from Harrm and kamilsa August 22, 2022 15:16
@codecov
Copy link

codecov bot commented Aug 22, 2022

Codecov Report

Merging #1326 (ef98862) into master (ba0b00a) will decrease coverage by 0.03%.
The diff coverage is 8.51%.

@@            Coverage Diff             @@
##           master    #1326      +/-   ##
==========================================
- Coverage   25.70%   25.67%   -0.04%     
==========================================
  Files         595      598       +3     
  Lines       21521    21566      +45     
  Branches    11169    11200      +31     
==========================================
+ Hits         5532     5536       +4     
- Misses      10878    10916      +38     
- Partials     5111     5114       +3     
Impacted Files Coverage Δ
core/api/service/state/requests/call.cpp 0.00% <0.00%> (ø)
core/api/service/state/requests/call.hpp 0.00% <0.00%> (ø)
core/api/service/state/state_api.hpp 100.00% <ø> (ø)
core/api/service/state/state_jrpc_processor.cpp 51.61% <0.00%> (-1.73%) ⬇️
core/injector/application_injector.cpp 34.67% <ø> (ø)
core/offchain/impl/offchain_worker_impl.cpp 0.00% <ø> (ø)
core/offchain/offchain_worker_factory.hpp 100.00% <ø> (ø)
core/runtime/binaryen/core_api_factory_impl.cpp 17.85% <ø> (ø)
core/runtime/common/executor.hpp 16.92% <0.00%> (ø)
...ore/runtime/runtime_api/impl/account_nonce_api.cpp 33.33% <ø> (ø)
... and 13 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

core/runtime/raw_executor.hpp Outdated Show resolved Hide resolved
Co-authored-by: kamilsa <kamilsa16@gmail.com>
@GaroRobe GaroRobe enabled auto-merge (squash) August 23, 2022 11:04
@GaroRobe GaroRobe merged commit 04cc63f into master Aug 23, 2022
@GaroRobe GaroRobe deleted the feature/state_call-api branch August 23, 2022 12:04
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.

Fix extrinsics status in polkadot js
3 participants