Add identity chain actions, switch from hex to base58, switch to borsh for signature, refactor identity chain #273
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📝 Description
Future PRs in this task: adding atomicity for surrealdb blockchain actions and rollback logic in case of errors
This PR refactors the identity chain to use in-module data objects and a fail-safe API to add blocks.
It also switches from hex to base58 encoding and to borsh for the signature.
It also re-implements changing identity fields to use an Option-based approach.
Relates to #140
✅ Checklist
Please ensure the following tasks are completed before requesting a review:
cargo fmt
.🚀 Changes Made
New Features:
Refactoring:
Other Changes:
💡 How to Test
Please provide clear instructions on how reviewers can test your changes:
🤝 Related Issues
List any related issues, pull requests, or discussions:
📋 Review Guidelines
Please focus on the following while reviewing: