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

Nodes contract: main implementation and tests #540

Closed
fbac opened this issue Feb 24, 2025 · 0 comments · Fixed by #524
Closed

Nodes contract: main implementation and tests #540

fbac opened this issue Feb 24, 2025 · 0 comments · Fixed by #524
Assignees

Comments

@fbac
Copy link
Collaborator

fbac commented Feb 24, 2025

No description provided.

@fbac fbac self-assigned this Feb 24, 2025
@fbac fbac moved this to In Progress in Decentralization Feb 24, 2025
fbac added a commit that referenced this issue Feb 24, 2025
Closes #540

- Added metadata to Nodes NFT.
- An EnumerableSet tracks the active nodes. Saves gas when calling the
contract to check for an active node.

Note: This is a breaking change. Go tests are not expected to pass.

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

- **New Features**
- Introduced an NFT-based registry interface that streamlines node
lifecycle management with secure node addition, property updates,
transfers, and batch operations.
- Launched an enhanced node registry contract featuring robust
role-based access control and comprehensive querying capabilities.

- **Style**
  - Applied minor formatting adjustments for improved consistency.

- **Chores**
- Optimized metadata handling, caching strategies, and deployment
scripts with stronger environment validations and streamlined deployment
data serialization.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->
@fbac fbac closed this as completed in #524 Feb 24, 2025
@github-project-automation github-project-automation bot moved this from In Progress to Done in Decentralization Feb 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant