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

Move payload generation to worker processes #612

Merged
merged 31 commits into from
May 18, 2023

Conversation

martinkersner
Copy link
Member

@martinkersner martinkersner commented May 11, 2023

Description

This PR moves a creation of transaction payload (to, gasLimit as well) to worker processes. Reporter service will be used by all worker processes. This will lead to easier reporter code management.

All services (VRF, Request-Response, and Data Feed) from now require registration of their reporters to Orakl Network permanent state through Orakl Network CLI (orakl-cli reporter insert). Reporters defined within .env file through PUBLIC_KEY and PRIVATE_KEY are ignored!

Others

This PR additionally changes publishing of @bisonai/orakl-contracts package. Previously, npm package was published when repository published a new release. From now, new npm package is generated when version is updated at package.json inside of contracts/ directory.

Published version 0.6.0.

Type of change

  • New feature (non-breaking change which adds functionality)

Checklist before requesting a review

  • I have performed a self-review of my code.
  • If it is a core feature, I have added thorough tests.

Deployment

  • Should publish npm package
  • Should publish Docker image

@martinkersner martinkersner self-assigned this May 11, 2023
@martinkersner martinkersner marked this pull request as ready for review May 17, 2023 14:07
@martinkersner martinkersner requested a review from KelvinThai May 17, 2023 14:07
@KelvinThai KelvinThai merged commit 88353e1 into master May 18, 2023
@KelvinThai KelvinThai deleted the i-483/feat/move-payload-generation-to-workers branch May 18, 2023 11:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Build a transaction payload at worker. Reporter just reports. Single reporter definition for all.
2 participants