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

cacti polkadot connector #1

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

cacti polkadot connector #1

wants to merge 2 commits into from

Conversation

AnmolBansalDEV
Copy link
Owner

Pull Request Requirements

  • Rebased onto upstream/main branch and squashed into single commit to help maintainers review it more efficient and to avoid spaghetti git commit graphs that obfuscate which commit did exactly what change, when and, why.
  • Have git sign off at the end of commit message to avoid being marked red. You can add -s flag when using git commit command. You may refer to this link for more information.
  • Follow the Commit Linting specification. You may refer to this link for more information.

Character Limit

  • Pull Request Title and Commit Subject must not exceed 72 characters (including spaces and special characters).
  • Commit Message per line must not exceed 80 characters (including spaces and special characters).

A Must Read for Beginners
For rebasing and squashing, here's a must read guide for beginners.

@jagpreetsinghsasan
Copy link
Collaborator

I can review it as it progress

@AnmolBansalDEV AnmolBansalDEV changed the title Lfx connector cacti polkadot connector Oct 23, 2023
@RafaelAPB
Copy link
Collaborator

Please also fix the vulnerabilities pointed out by CodeQL

@AnmolBansalDEV AnmolBansalDEV force-pushed the lfx-connector branch 3 times, most recently from a532989 to 0a57c36 Compare December 25, 2023 10:58
@AnmolBansalDEV AnmolBansalDEV force-pushed the lfx-connector branch 2 times, most recently from 17fb399 to fa16a90 Compare February 7, 2024 06:46
@AnmolBansalDEV AnmolBansalDEV force-pushed the lfx-connector branch 2 times, most recently from 73bd582 to 1927bd4 Compare February 14, 2024 21:23
@AnmolBansalDEV AnmolBansalDEV force-pushed the lfx-connector branch 2 times, most recently from e9241db to 5ae638f Compare February 25, 2024 07:08
@dependabot @petermetz
build(deps-dev): bump vite in /packages/cacti-ledger-browser-react
2dc0de8 
Bumps [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite) from 5.0.8 to 5.0.12.
- [Release notes](https://github.com/vitejs/vite/releases)
- [Changelog](https://github.com/vitejs/vite/blob/v5.0.12/packages/vite/CHANGELOG.md)
- [Commits](https://github.com/vitejs/vite/commits/v5.0.12/packages/vite)

---
updated-dependencies:
- dependency-name: vite
  dependency-type: direct:development
...

Co-authored-by: Peter Somogyvari <[email protected]>

Signed-off-by: dependabot[bot] <[email protected]>
Signed-off-by: Peter Somogyvari <[email protected]>
@petermetz petermetz force-pushed the lfx-connector branch 5 times, most recently from fb13476 to f8be5c2 Compare February 27, 2024 20:09
@AnmolBansalDEV @petermetz
feat(connector-polkadot): add connector pkg, openapi specs, test suite
48308ed 
Primary Changes
---------------
1. Created openapi specs for get-prometheus-exporter-metrics, get-transaction-info,
   get-raw-transaction, sign-raw-transaction, run-transaction, deploy-contract-ink,
   and invoke-contract endpoints
2. Created relevant types for the request as well as response for the above endpoints
3. Added generated code for these endpoints
4. Created connector class with functions to interact with the polkadot ledger
5. Created webservices to interact with the endpoint and the relevant class method
6. Created unit and integration testcases in jest to test base functionality of the connector

Secondary Changes
-----------------
1. Added an ink! contract for running the testcases
2. Added the polkadot connector to ci workflow
3. Created substrate.md to docs-cactus
4. Added the polkadot connector to tsconfig.json

Signed-off-by: Anmol Bansal <[email protected]>

=======================================================================

feat(polkadot): creation of polkadot AIO image

Primary Changes
---------------
1. Updated docker image to include multi-stage build
2. Added healthcheck to the image
3. Added Supervisord and removed unneccessary start script
4. Updated the Readme with relevant commands

Signed-off-by: Anmol Bansal <[email protected]>

=======================================================================

feat(polkadot): update substrate test tooling

Primary Changes
---------------
1. Added correct healthcheck for ledger container
2. Update the Substrate test ledger testcases

Signed-off-by: Anmol Bansal <[email protected]>

=======================================================================

feat(polkadot): creation of readme and architecture reference diagrams

Primary Changes
---------------
1. Added README.md for the connector
2. Added Architecture diagrams for the plugin

Secondary Changes
-----------------
1. Added the docker image for the polkadot connector

Fixes hyperledger/cacti#726
Fixes hyperledger/cacti#727
Fixes hyperledger/cacti#627

Co-authored-by: Peter Somogyvari <[email protected]>

Signed-off-by: Anmol Bansal <[email protected]>
Signed-off-by: Peter Somogyvari <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jagpreetsinghsasan jagpreetsinghsasan Awaiting requested review from jagpreetsinghsasan jagpreetsinghsasan is a code owner

@petermetz petermetz Awaiting requested review from petermetz petermetz is a code owner

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

Successfully merging this pull request may close these issues.
3 participants
@AnmolBansalDEV @jagpreetsinghsasan @RafaelAPB