-
Notifications
You must be signed in to change notification settings - Fork 286
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
feat(cactus-plugin-persistence-ethereum): add new persistence plugin #2259
Conversation
- Add GUI app to visualize ledger data stored in supabase by persistence plugins Closes: hyperledger-cacti#2264 Depends on: hyperledger-cacti#2259 Signed-off-by: Eryk Baranowski <eryk.baranowski@fujitsu.com>
packages/cactus-plugin-persistence-ethereum/src/main/typescript/plugin-persistence-ethereum.ts
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
...ce-ethereum/src/main/typescript/generated/openapi/typescript-axios/.openapi-generator-ignore
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@outSH Thanks for the changes! I added a few more asks because of the release that went out just yesterday made the dependency versions outdated.
Last thing: There are 2 commits now, one of them from the parent PR. I'm guessing if you do a rebase + conflict resolution it will go back down to 1 commit with that being just the one we want but in case it doesn't, please look into that as well.
This PR/issue depends on: |
- Add a new plugin for storing ledger data into a database (or any other storage in the future). - Add functional tests for plugin and data access layer operations. - Increase gas limit on openethereum contract to solve occasional issues in automatic tests. - Tests assume any postgres database, but for final deployment supabase is assumed. - Data fed by this plugin can later by visualized by a GUI application or analyzed directly. - Add new plugin tests to CI Depends on: hyperledger-cacti#2254 Depends on: hyperledger-cacti#2256 Signed-off-by: Michal Bajer <michal.bajer@fujitsu.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@outSH Thank you for the updates, LGTM!
-Add GUI app to visualize ledger data stored in supabase by persistence plugins Closes: hyperledger-cacti#2264 Depends on: hyperledger-cacti#2259 Signed-off-by: Eryk Baranowski eryk.baranowski@fujitsu.com
-Add GUI app to visualize ledger data stored in supabase by persistence plugins Closes: hyperledger-cacti#2264 Depends on: hyperledger-cacti#2259 Signed-off-by: Eryk Baranowski eryk.baranowski@fujitsu.com
1. Add a new plugin for storing hyperledger fabric ledger data into a database 2. Add functional tests for plugin and data access layer operations. operating with fabric-all-in-one docker composition Tests assume any postgres database, but for final deployment postgres or supabase is assumed. Data fed by this plugin can later be visualized by a GUI application or analyzed directly. ( right now the GUI will have switch to operate either with Ethereum ledger or Fabric ledger ) Also upgrade ts-node in the rood package.json because it was causing crashes for the scripts in the ./tools/ directory (such as the webpack bundle name validator script) Depends on: hyperledger-cacti#2259 Depends on: hyperledger-cacti#2265 Co-authored-by: Peter Somogyvari <peter.somogyvari@accenture.com> Signed-off-by: Barnaba Pawełczak <barnaba.pawelczak@fujitsu.com> Signed-off-by: Peter Somogyvari <peter.somogyvari@accenture.com>
1. Add a new plugin for storing hyperledger fabric ledger data into a database 2. Add functional tests for plugin and data access layer operations. operating with fabric-all-in-one docker composition Tests assume any postgres database, but for final deployment postgres or supabase is assumed. Data fed by this plugin can later be visualized by a GUI application or analyzed directly. ( right now the GUI will have switch to operate either with Ethereum ledger or Fabric ledger ) Also upgrade ts-node in the rood package.json because it was causing crashes for the scripts in the ./tools/ directory (such as the webpack bundle name validator script) Depends on: #2259 Depends on: #2265 Co-authored-by: Peter Somogyvari <peter.somogyvari@accenture.com> Signed-off-by: Barnaba Pawełczak <barnaba.pawelczak@fujitsu.com> Signed-off-by: Peter Somogyvari <peter.somogyvari@accenture.com>
1. Add a new plugin for storing hyperledger fabric ledger data into a database 2. Add functional tests for plugin and data access layer operations. operating with fabric-all-in-one docker composition Tests assume any postgres database, but for final deployment postgres or supabase is assumed. Data fed by this plugin can later be visualized by a GUI application or analyzed directly. ( right now the GUI will have switch to operate either with Ethereum ledger or Fabric ledger ) Also upgrade ts-node in the rood package.json because it was causing crashes for the scripts in the ./tools/ directory (such as the webpack bundle name validator script) Depends on: hyperledger-cacti#2259 Depends on: hyperledger-cacti#2265 Co-authored-by: Peter Somogyvari <peter.somogyvari@accenture.com> Signed-off-by: Barnaba Pawełczak <barnaba.pawelczak@fujitsu.com> Signed-off-by: Peter Somogyvari <peter.somogyvari@accenture.com>
1. Add a new plugin for storing hyperledger fabric ledger data into a database 2. Add functional tests for plugin and data access layer operations. operating with fabric-all-in-one docker composition Tests assume any postgres database, but for final deployment postgres or supabase is assumed. Data fed by this plugin can later be visualized by a GUI application or analyzed directly. ( right now the GUI will have switch to operate either with Ethereum ledger or Fabric ledger ) Also upgrade ts-node in the rood package.json because it was causing crashes for the scripts in the ./tools/ directory (such as the webpack bundle name validator script) Depends on: hyperledger-cacti#2259 Depends on: hyperledger-cacti#2265 Co-authored-by: Peter Somogyvari <peter.somogyvari@accenture.com> Signed-off-by: Barnaba Pawełczak <barnaba.pawelczak@fujitsu.com> Signed-off-by: Peter Somogyvari <peter.somogyvari@accenture.com>
Depends on #2254
Depends on #2256
Signed-off-by: Michal Bajer michal.bajer@fujitsu.com