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

milestone 1: Add type registry and test #327

Merged
merged 5 commits into from
Mar 15, 2023
Merged

milestone 1: Add type registry and test #327

merged 5 commits into from
Mar 15, 2023

Conversation

cdamian
Copy link
Contributor

@cdamian cdamian commented Feb 2, 2023 •
edited
Loading

@codecov
Copy link

codecov bot commented Feb 2, 2023 •
edited
Loading

Codecov Report

❗ No coverage uploaded for pull request base (event-parsing-v2@d742d37). Click here to learn what that means.
The diff coverage is n/a.

@@                 Coverage Diff                 @@
##             event-parsing-v2     #327   +/-   ##
===================================================
  Coverage                    ?   63.63%           
===================================================
  Files                       ?        1           
  Lines                       ?       11           
  Branches                    ?        0           
===================================================
  Hits                        ?        7           
  Misses                      ?        2           
  Partials                    ?        2

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

mikiquantum
mikiquantum reviewed Feb 6, 2023
View reviewed changes
Copy link
Contributor

@mikiquantum mikiquantum left a comment

Choose a reason for hiding this comment

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

This looks great, thanks @cdamian!

events/registry.go Outdated Show resolved Hide resolved
events/registry.go Outdated Show resolved Hide resolved
events/registry.go Outdated Show resolved Hide resolved
events/registry_test.go Outdated Show resolved Hide resolved
@cdamian cdamian changed the title milestone: Add type registry and test milestone 1: Add type registry and test Feb 9, 2023
mikiquantum
mikiquantum approved these changes Feb 9, 2023
View reviewed changes
Copy link
Contributor

@mikiquantum mikiquantum left a comment

Choose a reason for hiding this comment

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

Looks good!

events/registry_v2.go Outdated Show resolved Hide resolved
@cdamian cdamian force-pushed the events-parsing-v2-milestone-1 branch from 81ebbd8 to aa507a8 Compare February 10, 2023 12:36
@cdamian cdamian force-pushed the events-parsing-v2-milestone-1 branch from 19aa300 to a6e538c Compare March 15, 2023 13:28
@cdamian cdamian marked this pull request as ready for review March 15, 2023 13:32
@cdamian cdamian merged commit 772451a into event-parsing-v2 Mar 15, 2023
@cdamian cdamian deleted the events-parsing-v2-milestone-1 branch March 15, 2023 13:33
@cdamian cdamian mentioned this pull request Mar 15, 2023
Merged
cdamian added a commit that referenced this pull request Jul 4, 2023
@cdamian
milestone 1: Add type registry and test (#327)
846c0bf 
* milestone: Add type registry and test

* milestone1: Add metadatas from multiple chains and docker file for tests

* registry: Add methods to create call and error registries

* dockerfile: Fix test Dockerfile

* registry: Add more tests
cdamian added a commit that referenced this pull request Jul 5, 2023
@cdamian @mikiquantum @niklabh
Add events and extrinsics parsing logic based on metadata (#326)
20eecd5 
* events: Add initial metadata parsing logic

* events: Add override decoding funcs, add support for primitive arrays

* events: Extract decode type def

* events: Add event parsing test script, provide more details in error messages

* events: Add variant support for vectors

* milestone 1: Add type registry and test (#327)

* milestone: Add type registry and test

* milestone1: Add metadatas from multiple chains and docker file for tests

* registry: Add methods to create call and error registries

* dockerfile: Fix test Dockerfile

* registry: Add more tests

* Events parsing v2 milestone 2 (#338)

* milestone: Add type registry and test

* milestone1: Add metadatas from multiple chains and docker file for tests

* events: Add decoders for each event field type

* registry: Add decoders

* registry: Add retryable executor and use it in parser

* registry: Add mocks and more tests

* registry: Use BitVec when parsing bit sequences

* test: Add more mocks and tests

* registry: Update field name retrieval

* registry-test: Get headers instead of blocks

* Add generic chain RPC, more tests, and Dockerfiles for the 2nd milestone

* make: Add container name

* chain: Add constructor for default chain

* registry: Change field separator to

* parser: Add DefaultExtrinsicParser and DefaultExtrinsic

* retriever: Add DefaultExtrinsicRetriever and adjust tests

* rpc: Add more comments to default entities

* retriever: Enable all live tests

* Add registry readme (#351)

* Fix docker run for w3f milestone 2 tests (#353)

* Address review milestone 2 (#355)

* add configurable event threshold

* Add natural language desc on event parsing

* make: Remove grant related targets

---------

Co-authored-by: Miguel Hervas <miguel.hervas.lazaro@gmail.com>
Co-authored-by: Nikhil Ranjan <niklabh811@gmail.com>
kroos47 pushed a commit to availproject/go-substrate-rpc-client that referenced this pull request Sep 26, 2023
@cdamian @mikiquantum
@niklabh @kroos47
Add events and extrinsics parsing logic based on metadata (centrifuge…
25b2141 
…#326)

* events: Add initial metadata parsing logic

* events: Add override decoding funcs, add support for primitive arrays

* events: Extract decode type def

* events: Add event parsing test script, provide more details in error messages

* events: Add variant support for vectors

* milestone 1: Add type registry and test (centrifuge#327)

* milestone: Add type registry and test

* milestone1: Add metadatas from multiple chains and docker file for tests

* registry: Add methods to create call and error registries

* dockerfile: Fix test Dockerfile

* registry: Add more tests

* Events parsing v2 milestone 2 (centrifuge#338)

* milestone: Add type registry and test

* milestone1: Add metadatas from multiple chains and docker file for tests

* events: Add decoders for each event field type

* registry: Add decoders

* registry: Add retryable executor and use it in parser

* registry: Add mocks and more tests

* registry: Use BitVec when parsing bit sequences

* test: Add more mocks and tests

* registry: Update field name retrieval

* registry-test: Get headers instead of blocks

* Add generic chain RPC, more tests, and Dockerfiles for the 2nd milestone

* make: Add container name

* chain: Add constructor for default chain

* registry: Change field separator to

* parser: Add DefaultExtrinsicParser and DefaultExtrinsic

* retriever: Add DefaultExtrinsicRetriever and adjust tests

* rpc: Add more comments to default entities

* retriever: Enable all live tests

* Add registry readme (centrifuge#351)

* Fix docker run for w3f milestone 2 tests (centrifuge#353)

* Address review milestone 2 (centrifuge#355)

* add configurable event threshold

* Add natural language desc on event parsing

* make: Remove grant related targets

---------

Co-authored-by: Miguel Hervas <miguel.hervas.lazaro@gmail.com>
Co-authored-by: Nikhil Ranjan <niklabh811@gmail.com>
MRamanenkau added a commit to Cerebellum-Network/go-substrate-rpc-client that referenced this pull request Feb 7, 2024
@MRamanenkau @cdamian
@1llusion1st @DragonDmoney @leovct @mikiquantum @niklabh @xrash
Feature/sync with upstream (#11)
a79ce14 
* client: Expose close function (centrifuge#311)

* fix of account-info structure (centrifuge#309)

Co-authored-by: Andriy <andriy.vasyltsiv@gmail.com>

* types: Update the dispatch info weight (centrifuge#323)

* types: Update anonymous proxy created event (centrifuge#324)

* change retarded uint8 to uint16 for networkid (centrifuge#339)

* fix: typos in `README` (centrifuge#348)

* fix: typos in `README`

* doc: add missing section

* Update event types (centrifuge#349)

* event: Update treasury events

* types: Add Orml Asset Registry events

* types: Update CurrencyID

* events: Adjust CustomMetadata for Orml events (centrifuge#354)

* Add events and extrinsics parsing logic based on metadata (centrifuge#326)

* events: Add initial metadata parsing logic

* events: Add override decoding funcs, add support for primitive arrays

* events: Extract decode type def

* events: Add event parsing test script, provide more details in error messages

* events: Add variant support for vectors

* milestone 1: Add type registry and test (centrifuge#327)

* milestone: Add type registry and test

* milestone1: Add metadatas from multiple chains and docker file for tests

* registry: Add methods to create call and error registries

* dockerfile: Fix test Dockerfile

* registry: Add more tests

* Events parsing v2 milestone 2 (centrifuge#338)

* milestone: Add type registry and test

* milestone1: Add metadatas from multiple chains and docker file for tests

* events: Add decoders for each event field type

* registry: Add decoders

* registry: Add retryable executor and use it in parser

* registry: Add mocks and more tests

* registry: Use BitVec when parsing bit sequences

* test: Add more mocks and tests

* registry: Update field name retrieval

* registry-test: Get headers instead of blocks

* Add generic chain RPC, more tests, and Dockerfiles for the 2nd milestone

* make: Add container name

* chain: Add constructor for default chain

* registry: Change field separator to

* parser: Add DefaultExtrinsicParser and DefaultExtrinsic

* retriever: Add DefaultExtrinsicRetriever and adjust tests

* rpc: Add more comments to default entities

* retriever: Enable all live tests

* Add registry readme (centrifuge#351)

* Fix docker run for w3f milestone 2 tests (centrifuge#353)

* Address review milestone 2 (centrifuge#355)

* add configurable event threshold

* Add natural language desc on event parsing

* make: Remove grant related targets

---------

Co-authored-by: Miguel Hervas <miguel.hervas.lazaro@gmail.com>
Co-authored-by: Nikhil Ranjan <niklabh811@gmail.com>

* events: Add ORML tokens events (centrifuge#358)

* registry: Add overrides for field decoders (centrifuge#359)

* Change fields format to slice (centrifuge#360)

* registry: Add decoded fields as slice, add helper methods for retrieving values

* types: Remove unnecessary tests

* registry: Update return type of composite decoder to DecodedFields

* registry: Add ErrorID for ErrorRegistry, remove encode/decode for DecodedField

* registry: Add more tests for overrides

* registry: Reset storages when creating registries

* docs: Update godoc link (centrifuge#364)

* Polkadot update v0.9.43 (centrifuge#366)

* types: Update AccountInfo

* tests: Update tests

* client: Add CallContext to Client interface

* deps: Update go version, polkadot docker image

* lint: Obey

* deps: Update subkey to v2

* address: Return account ID creation error

* Remove extraneous output (centrifuge#343)

* Fix compilation errors

* Update to v8

---------

Co-authored-by: Cosmin Damian <17934949+cdamian@users.noreply.github.com>
Co-authored-by: 1llusion1st <98103979+1llusion1st@users.noreply.github.com>
Co-authored-by: Andriy <andriy.vasyltsiv@gmail.com>
Co-authored-by: Pierre <24465444+DragonDmoney@users.noreply.github.com>
Co-authored-by: Léo Vincent <28714795+leovct@users.noreply.github.com>
Co-authored-by: Miguel Hervas <miguel.hervas.lazaro@gmail.com>
Co-authored-by: Nikhil Ranjan <niklabh811@gmail.com>
Co-authored-by: Felipe <felipe@xra.sh>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mikiquantum mikiquantum mikiquantum approved these changes

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

Successfully merging this pull request may close these issues.
2 participants
@cdamian @mikiquantum