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

Update README.md #183

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sergiuchuckmisha
Copy link

current documentation about creation of new Exonum.Transaction with light client won't work because of wrong names of service-id and method-id fields and needs to be updated

This workable .js code may be considered as an example:
https://github.com/exonum/exonum/blob/v1.0.0-rc.1/examples/cryptocurrency-advanced/frontend/src/plugins/blockchain.js#L14
https://github.com/exonum/exonum/blob/v1.0.0-rc.1/examples/cryptocurrency-advanced/frontend/src/plugins/blockchain.js#L15

current documentation about creation of `new Exonum.Transaction` with light client won't work because of wrong names of `service-id` and `method-id` fields and needs to be updated

This workable .js code may be considered as an example: 
https://github.com/exonum/exonum/blob/v1.0.0-rc.1/examples/cryptocurrency-advanced/frontend/src/plugins/blockchain.js#L14
https://github.com/exonum/exonum/blob/v1.0.0-rc.1/examples/cryptocurrency-advanced/frontend/src/plugins/blockchain.js#L15
@coveralls
Copy link

Coverage Status

Coverage remained the same at 93.203% when pulling 909f2d1 on sergiuchuckmisha:patch-1 into c6ca1c0 on exonum:master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants