Skip to content

refactor(contract_manager): restructure contract_manager #2711

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

Merged
merged 11 commits into from
May 27, 2025

Conversation

cctdaniel
Copy link
Contributor

@cctdaniel cctdaniel commented May 21, 2025

Summary

Restructured the contract manager to core and node for browser import, converted yaml config to json for export.

Rationale

These changes were necessary to:

  1. Allow browser import of contract manager components that don't depend on node (e.g., fs)
  2. Enable other packages to use the exported json config files
  3. Improve modularity by separating core and node-specific functionality

How has this been tested?

  • Current tests cover my changes
  • Added new tests
  • Manually tested the code

Verified changes by:

  • Testing browser imports of core components
  • Validating json config exports in dependent packages
  • Ensuring all yaml to json conversions maintain data integrity

Copy link

vercel bot commented May 21, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
api-reference ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 27, 2025 5:31am
component-library ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 27, 2025 5:31am
developer-hub ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 27, 2025 5:31am
entropy-debugger ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 27, 2025 5:31am
entropy-explorer ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 27, 2025 5:31am
insights ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 27, 2025 5:31am
proposals ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 27, 2025 5:31am
staking ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 27, 2025 5:31am

@cctdaniel cctdaniel force-pushed the contract-manager-refactor branch from c2d89dc to 53efeae Compare May 21, 2025 06:01
@vercel vercel bot temporarily deployed to Preview – entropy-debugger May 21, 2025 06:37 Inactive
@vercel vercel bot temporarily deployed to Preview – insights May 21, 2025 06:37 Inactive
@vercel vercel bot temporarily deployed to Preview – proposals May 21, 2025 06:37 Inactive
@vercel vercel bot temporarily deployed to Preview – entropy-explorer May 21, 2025 06:37 Inactive
@vercel vercel bot temporarily deployed to Preview – staking May 21, 2025 06:37 Inactive
@vercel vercel bot temporarily deployed to Preview – developer-hub May 21, 2025 06:37 Inactive
@vercel vercel bot temporarily deployed to Preview – api-reference May 21, 2025 06:37 Inactive
@vercel vercel bot temporarily deployed to Preview – component-library May 21, 2025 06:37 Inactive
@vercel vercel bot temporarily deployed to Preview – component-library May 21, 2025 06:43 Inactive
@vercel vercel bot temporarily deployed to Preview – staking May 21, 2025 06:43 Inactive
@vercel vercel bot temporarily deployed to Preview – insights May 21, 2025 06:43 Inactive
@vercel vercel bot temporarily deployed to Preview – api-reference May 21, 2025 06:43 Inactive
@vercel vercel bot temporarily deployed to Preview – entropy-explorer May 21, 2025 06:43 Inactive
@vercel vercel bot temporarily deployed to Preview – entropy-debugger May 21, 2025 06:43 Inactive
@vercel vercel bot temporarily deployed to Preview – proposals May 21, 2025 06:43 Inactive
@vercel vercel bot temporarily deployed to Preview – developer-hub May 21, 2025 06:43 Inactive
@vercel vercel bot temporarily deployed to Preview – api-reference May 27, 2025 03:54 Inactive
@vercel vercel bot temporarily deployed to Preview – entropy-explorer May 27, 2025 03:54 Inactive
@vercel vercel bot temporarily deployed to Preview – insights May 27, 2025 03:54 Inactive
@vercel vercel bot temporarily deployed to Preview – component-library May 27, 2025 03:54 Inactive
@vercel vercel bot temporarily deployed to Preview – developer-hub May 27, 2025 03:54 Inactive
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.

2 participants