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

feat: added RegistryGenerator service class to export ERC token contracts #1050

Conversation

quiet-node
Copy link
Member

@quiet-node quiet-node commented Dec 5, 2024

Description:
This PR adds the RegistryGenerator service class, which exports ERC token contracts to JSON files and writes them to disk. Depending on the specified HEDERA_NETWORK value, the registry will create a dedicated folder for the JSON files.

Additionally, this PR removes the MIRROR_NODE_URL variable, as it is no longer needed.

Related issue(s):

Fixes #1040

Notes for reviewer:
This branch is based on PR #1048; therefore, the changes in PR #1048 are reflected in this PR.

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

@quiet-node quiet-node added the enhancement New feature or request label Dec 5, 2024
@quiet-node quiet-node added this to the 0.11.0 milestone Dec 5, 2024
@quiet-node quiet-node self-assigned this Dec 5, 2024
@quiet-node quiet-node requested a review from a team as a code owner December 5, 2024 02:54
@quiet-node quiet-node force-pushed the 1040-erc-registry-add-registrygenerator-service-class-to-handle-exporting-erc-contracts-into-json-files branch from 9064b8f to a7a1606 Compare December 5, 2024 02:54
Copy link

github-actions bot commented Dec 5, 2024

Test Results

 16 files  ±0   83 suites  ±0   12m 7s ⏱️ ±0s
294 tests ±0  288 ✅ ±0  6 💤 ±0  0 ❌ ±0 
303 runs  ±0  296 ✅ ±0  7 💤 ±0  0 ❌ ±0 

Results for commit c13db76. ± Comparison against base commit 3c9bdf6.

♻️ This comment has been updated with latest results.

natanasow
natanasow previously approved these changes Dec 5, 2024
Copy link
Contributor

@natanasow natanasow left a comment

Choose a reason for hiding this comment

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

LGTM.

@quiet-node quiet-node force-pushed the 1040-erc-registry-add-registrygenerator-service-class-to-handle-exporting-erc-contracts-into-json-files branch from a7a1606 to e884eda Compare December 5, 2024 15:26
@quiet-node quiet-node dismissed natanasow’s stale review December 5, 2024 15:27

The merge-base changed after approval.

@quiet-node quiet-node force-pushed the 1040-erc-registry-add-registrygenerator-service-class-to-handle-exporting-erc-contracts-into-json-files branch from e884eda to f69d641 Compare December 5, 2024 15:27
Signed-off-by: Logan Nguyen <[email protected]>
@quiet-node quiet-node merged commit d4dc991 into main Dec 5, 2024
30 checks passed
@quiet-node quiet-node deleted the 1040-erc-registry-add-registrygenerator-service-class-to-handle-exporting-erc-contracts-into-json-files branch December 5, 2024 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ERC Registry] Add RegistryGenerator service class to handle exporting ERC contracts into JSON files
2 participants