-
Notifications
You must be signed in to change notification settings - Fork 37
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
Support named networks, Amoy blockchain and Banksia network #412
Conversation
…tworkDefinition` for explicit metadata relating to Verida networks.
}, | ||
} | ||
|
||
export const BLOCKCHAIN_CHAINIDS: Record<BlockchainAnchor, string> = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The blockchain hex ids are defined in two places, risk of unsync and confusion on why one to use.
See other comment regarding a blockchain definition object.
packages/vda-common/src/contract.ts
Outdated
|
||
/** | ||
* @todo Deprecate in favour of `NETWORK_DEFINITIONS` | ||
*/ | ||
export const CONTRACT_ADDRESS : Record<CONTRACT_NAMES, Record<string, string | null>> = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This deprecated CONTRACT_ADDRESS
still holds info not in the network defintiions, such as for the DID registry addresses.
I gues we should have a similar object for the DID VDA, gathering the registry address, its blockchain and other related information (name registry?).
- @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected]
- Updated `proof` related code - Updated test script - Added `Test.md` - Updated `.env.example` file
- Updated `.env.example` file
- Added `Test.md` file - Updated .env.example file
- Added `Test.md` file - Updated .env.example file
- Added `Test.md` file - Updated the .env.example file
- Added `Test.md` file - Updated `.env.example` file
- @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected]
- @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected]
- @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected] - @verida/[email protected]
- Separte `stake_and_lock.test.ts` from the `user_write.test.ts` [vda-common-test] Added `owner()` function to the `ERC20Manager`
- Added `owner()` function - Update `addInitialData()`: general users can run the tests
Have you addressed this feedback?
Also see my other code comments. |
v4 Release NotesBreaking changesSeparating DID blockchain from Verida Network blockchainThe protocol previously had the concept of a Verida Network, which was anchored to a single blockchain. In order to maximize flexibility and handle blockchains going offline (such as Mumbai which has been shut down), the protocol now separates the blockchain used to anchor a The protocol supports:
DID's can now be anchored to any blockchain (or linked with any other DID method). The storage nodes for an application context are stored in the DID document, which has a reference to the Verida network associated with the storage nodes. In this way, it's possible to have a single DID document that is used across multiple Verida networks. As a result, the way a connection is established has changed.
Storage node authenticationAuthentication between storage nodes and clients requires signing a consent message (containing a generated nonce) to ensure the DID controller is the one requesting access to the encrypted storage node data. For enhanced flexibility and security, the expected signing key has changed from the master DID account key to the context signing key. Other key changes:
|
Closes #411 #420 #421