-
Notifications
You must be signed in to change notification settings - Fork 16
feat(ServiceProviderRegistry): KeyValue productData #356
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
Conversation
* randIndex * update _encodeDataSetCreateData * add failing randIndex test * fix test * always test the fallback methods * test rand 256 * use bigint literal * randU256 * fallbackRandU256 * export rand functions in index.ts
* fix: Use clientAddress for next dataset ID (#299) fix: resolves bug of using client address instead of signer for datsetId lookup * chore(master): release synapse-sdk 0.31.1 (#300) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> * update abi and fix tests * remove cdnEndEpoch from type * Update packages/synapse-sdk/wagmi.config.ts Co-authored-by: Rod Vagg <[email protected]> --------- Co-authored-by: Shashank Trivedi <[email protected]> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Rod Vagg <[email protected]>
chore: update for M3 contracts chore: update for M3 contracts
Co-authored-by: Srdjan <[email protected]>
Signed-off-by: Jakub Sztandera <[email protected]>
* feat: create low-level create-and-add API Signed-off-by: Jakub Sztandera <[email protected]> * add doc strings Signed-off-by: Jakub Sztandera <[email protected]> --------- Signed-off-by: Jakub Sztandera <[email protected]>
core updated to new contracts
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
synapse-dev | adb3820 | Commit Preview URL Branch Preview URL |
Oct 27 2025, 11:46 PM |
|
line count is inflated because wagmi sorts the abis by name, and the name of the Payments contract changed. |
| capabilityValues.push('0x01') | ||
| } | ||
| capabilityKeys.push('storagePricePerTibPerDay') | ||
| capabilityValues.push(`0x${pdpOffering.storagePricePerTibPerDay.toString(16)}`) |
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 won't work because the bytes values need to be of even length.
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.
I will use bytesToHex(toBytes
|
I'm done for today. Will resume this tomorrow. |
| */ | ||
| async getPDPService(providerId: number): Promise<PDPServiceInfo | null> { | ||
| try { | ||
| //try { |
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 is the main reason I haven't ran lint yet. Hoping to keep this removed until the tests pass, as otherwise it might mask some errors with the new code.
|
FYI Rod took it over: #361 |
This was not completed in #361. I presume it does not work. |
Reviewers @hugomrdias @Kubuxu @rvagg
Changes
Hextype in some places