cacti icon indicating copy to clipboard operation
cacti copied to clipboard

refactor(cc-tx-visualization): capture transactional data with RxJS #3100

Open brunoffmateus opened this issue 1 year ago • 3 comments

The plugin now utilizes RxJS instead of RabbitMQ in transaction monitoring. ReplaySubjects store and emit observed transactional data to subscribers.

brunoffmateus avatar Apr 15 '24 18:04 brunoffmateus

Please rebase with the latest version of satp-dev

RafaelAPB avatar Apr 16 '24 15:04 RafaelAPB

@brunoffmateus please don't forget to add me as a maintainer for this package (see https://github.com/brunoffmateus/blockchain-integration-framework/blob/satp-3100-cctx/CODEOWNERS).

Please also rename the plugin to cactus-plugin-ccmodel-hepheastus

RafaelAPB avatar Apr 24 '24 17:04 RafaelAPB

@brunoffmateus Looks much better. Please rename the plugin too

RafaelAPB avatar Apr 25 '24 07:04 RafaelAPB

️✅ There are no secrets present in this pull request anymore.

If these secrets were true positive and are still valid, we highly recommend you to revoke them. Once a secret has been leaked into a git repository, you should consider it compromised, even if it was deleted immediately. Find here more information about risks.


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

gitguardian[bot] avatar May 27 '24 13:05 gitguardian[bot]

@brunoffmateus Please squash the commits and fix the merge conflicts and make sure to rebase onto upstream main.

Also, the PR currently looks HUGE so if it's OK I'd love to help you break it down to smaller PRs.

As discussed, this will merge with satp-dev. When it eventually goes to main, I'll be sure to ask for your help!

brunoffmateus avatar May 31 '24 10:05 brunoffmateus

@brunoffmateus Please squash the commits and fix the merge conflicts and make sure to rebase onto upstream main. Also, the PR currently looks HUGE so if it's OK I'd love to help you break it down to smaller PRs.

As discussed, this will merge with satp-dev. When it eventually goes to main, I'll be sure to ask for your help!

@brunoffmateus Oh yes, sorry, I keep assuming that the PR is against main.

petermetz avatar May 31 '24 19:05 petermetz