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

Implement Trust Registry and introduce on-chain did:key support #147

Merged
merged 20 commits into from
Dec 29, 2023

Conversation

olegnn
Copy link
Collaborator

@olegnn olegnn commented Oct 19, 2023

No description provided.

@olegnn olegnn force-pushed the trust-registry-did-method-key branch from 474a7cf to a07a231 Compare October 19, 2023 13:42
@olegnn olegnn force-pushed the trust-registry-did-method-key branch 2 times, most recently from bf8708d to be46409 Compare October 19, 2023 16:58
@olegnn olegnn force-pushed the trust-registry-did-method-key branch from be46409 to d338ef1 Compare October 19, 2023 17:09
@olegnn olegnn force-pushed the trust-registry-did-method-key branch 4 times, most recently from 9b8c4b7 to 5c0388b Compare October 22, 2023 15:22
@olegnn olegnn force-pushed the trust-registry-did-method-key branch from 5c0388b to cc63dc8 Compare October 22, 2023 15:27
/// Adds a new schema metadata entry (entries).
/// The DID signature signer must be the `Convener` owning this Trust Registry.
#[pallet::weight(SubstrateWeight::<T>::add_schema_metadata(add_schema_metadata, sig))]
pub fn add_schema_metadata(
Copy link
Member

Choose a reason for hiding this comment

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

Is add_schema_metadata supposed to replace existing metadata? If yes, then it should be called set_schema_metadata

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It's not supposed

@lovesh lovesh force-pushed the trust-registry-did-method-key branch from 0c77c00 to 1a8349d Compare October 27, 2023 16:16
@lovesh lovesh force-pushed the trust-registry-did-method-key branch from 1a8349d to 664ac4d Compare October 27, 2023 16:23
pallets/core/src/util/batch_update.rs Show resolved Hide resolved
pallets/core/src/util/batch_update.rs Show resolved Hide resolved
pallets/core/src/util/batch_update.rs Outdated Show resolved Hide resolved
pallets/core/src/util/batch_update.rs Show resolved Hide resolved
@olegnn olegnn force-pushed the trust-registry-did-method-key branch from 73dfd50 to 97f4249 Compare October 27, 2023 18:31
@olegnn olegnn merged commit 7b53d68 into master Dec 29, 2023
4 checks passed
@olegnn olegnn deleted the trust-registry-did-method-key branch January 26, 2024 06:40
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