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

feat: include from param in bucket & blob methods #81

Merged
merged 2 commits into from
Feb 17, 2025

Conversation

dtbuchholz
Copy link
Contributor

@dtbuchholz dtbuchholz commented Feb 17, 2025

Closes #80

@dtbuchholz dtbuchholz requested review from ukstv and stbrody February 17, 2025 16:56
@dtbuchholz
Copy link
Contributor Author

note: i tested this against 793cc21f68d5703d8ba35fb73f31c2d1acd24d6f from recallnet/ipc#524.

@dtbuchholz dtbuchholz merged commit a87ac10 into main Feb 17, 2025
8 checks passed
@dtbuchholz dtbuchholz deleted the dtb/include-from-param branch February 17, 2025 18:01
dtbuchholz added a commit to recallnet/js-recall that referenced this pull request Feb 19, 2025
## Summary

Adds the new required `from` field in bucket methods for adding or
deleting objects via recallnet/contracts#81.
Also updates contracts and RPCs for the latest testnet. Closes
#51.

## Details
- Updates the `contracts` package with the latest changes, which alter
the ABI for `addObject` and `deleteObject` and add a new
`IMachineFacade` abi.
- Updates `network-constants` with new contracts and RPCs
- Includes the new `from` param in `sdk`, `sdkx`, and `studio`
- Removes events for wrappers, and only uses `IMachineFacade` abi
- Fix tests for `fvm`, `sdk`
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.

Add from param in blob and bucket methods
2 participants