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

Integrate supplementary Protocol Buffers (protobuf) parameters into the JS client to augment the extensibility and functionality of the data model within the designated oneof constructs, ensuring a more comprehensive representation and utilization of transaction related information #185

Open
wants to merge 22 commits into
base: main
Choose a base branch
from

Conversation

baziorek
Copy link
Member

This is PR with changes from: #181

Incorporate additional Protocol Buffers (protobuf) parameters into the JS client, enhancing the extensibility and functionality of the data model within the specified 'oneof' constructs. This integration ensures a more comprehensive representation and utilization of transaction-related information, thereby expanding the capabilities and flexibility of the system.

Those changes are connected with: hyperledger/iroha#4003 - this is part of internship project:
https://wiki.hyperledger.org/pages/viewpage.action?pageId=80778614

dependabot bot and others added 21 commits June 9, 2021 01:28
Bumps [glob-parent](https://github.com/gulpjs/glob-parent) from 5.1.1 to 5.1.2.
- [Release notes](https://github.com/gulpjs/glob-parent/releases)
- [Changelog](https://github.com/gulpjs/glob-parent/blob/main/CHANGELOG.md)
- [Commits](gulpjs/glob-parent@v5.1.1...v5.1.2)

---
updated-dependencies:
- dependency-name: glob-parent
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
…_yarn/glob-parent-5.1.2

Bump glob-parent from 5.1.1 to 5.1.2
* extended tx pagination meta queries added

Signed-off-by: Piotr Pawlowski <ppiotru@gmail.com>

* semicolon removed

Signed-off-by: Piotr Pawlowski <ppiotru@gmail.com>

* proto donwgraded

Signed-off-by: Piotr Pawlowski <ppiotru@gmail.com>

* example changed, args ordering updated, Pending txs updated

Signed-off-by: Piotr Pawlowski <ppiotru@gmail.com>

* params description added

Signed-off-by: Piotr Pawlowski <ppiotru@gmail.com>

* setField fixed

Signed-off-by: Piotr Pawlowski <ppiotru@gmail.com>

* version fixed

Signed-off-by: Piotr Pawlowski <ppiotru@gmail.com>

* version fixed

Signed-off-by: Piotr Pawlowski <ppiotru@gmail.com>

* yarn.lock fixed

Signed-off-by: Piotr Pawlowski <ppiotru@gmail.com>

* fix

Signed-off-by: Piotr Pawlowski <ppiotru@gmail.com>
Signed-off-by: Travis Payne <travis.payne@accenture.com>
…d-status

Adding tx hash and status to sendTransactions return
Signed-off-by: Stefan Popov <stefanpopov@list.ru>
Signed-off-by: Sara <lira.lemur@gmail.com>
Signed-off-by: yashrajdesai <yashrajdesai30@gmail.com>
Signed-off-by: yashrajdesai <yashrajdesai30@gmail.com>
Signed-off-by: yashrajdesai <yashrajdesai30@gmail.com>
…ontract

feat: Add smart-contract example
Signed-off-by: yashrajdesai <yashrajdesai30@gmail.com>
Signed-off-by: safinsaf <safinsaft@gmail.com>
Signed-off-by: Aleksandr Petrosyan <a-p-petrosyan@yandex.ru>
Signed-off-by: Ry Jones <ry@linux.com>
Signed-off-by: omahs <73983677+omahs@users.noreply.github.com>
Co-authored-by: 0x009922 <a.marcius26@gmail.com>
…age communication, with corresponding handling capabilities in Iroha CLI.

Signed-off-by: dominious1 <piotr567@proton.me>
@baziorek
Copy link
Member Author

This code should be tested by somebody who knows TypedScript (or rather is fluent with TypedScript). There should be made similar steps as for iroha-python: hyperledger/iroha-python#157 (comment)

  1. First of all server must be running. Here is instruction how to build its by its own: https://wiki.hyperledger.org/pages/viewpage.action?pageId=98731447 (official docker images are quite old, but required changes are new)
  2. The library from the branch should be installed git clone https://github.com/baziorek/iroha-javascript.git -b baziorek:proto_from_tomasz (not from official package repository)
  3. Then client must be prepared:
    1. Here is example code: https://github.com/hyperledger/iroha-javascript/blob/main/example/index.ts
    2. In the code we need only: 2 commands: https://iroha.readthedocs.io/en/main/develop/api/commands.html#add-asset-quantity and https://iroha.readthedocs.io/en/main/develop/api/commands.html#subtract-asset-quantity (they are not used in example), but with extra parameter "description". The next thing is to query transactions (it is in the example): https://github.com/baziorek/iroha-javascript/blob/968bd48c87de614da9a50b6b86dc839b274ce9a7/example/index.ts#L94
  4. If we see that newly introduced parameter "description" is available - it means that the PR is correct and can be merged.

Copy link
Contributor

@6r1d 6r1d left a comment

Choose a reason for hiding this comment

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

LGTM

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

Successfully merging this pull request may close these issues.

None yet