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.

Sign up for GitHub

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

Jump to bottom

refactor(sdk): sdk cleanup #909

Merged
merged 1 commit into from
Feb 5, 2025
Merged

refactor(sdk): sdk cleanup #909

merged 1 commit into from
Feb 5, 2025

Conversation

alexfreska
Copy link
Member

@alexfreska alexfreska commented Feb 3, 2025 •
edited
Loading

This PR adds methods to SDK required for sending v2 transactions in walletd.

  • Added v2TransactionInputSigHash and signHash methods to API.
  • Improved the accuracy of all return types.

@vercel Vercel
Copy link

vercel bot commented Feb 3, 2025 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

5 Skipped Deployments
Name Status Preview Comments Updated (UTC)
explorer ⬜️ Ignored (Inspect) Visit Preview Feb 5, 2025 2:04pm
explorer-zen ⬜️ Ignored (Inspect) Visit Preview Feb 5, 2025 2:04pm
hostd ⬜️ Ignored (Inspect) Visit Preview Feb 5, 2025 2:04pm
renterd ⬜️ Ignored (Inspect) Visit Preview Feb 5, 2025 2:04pm
website ⬜️ Ignored (Inspect) Visit Preview Feb 5, 2025 2:04pm

@changeset-bot changeset-bot
Copy link

changeset-bot bot commented Feb 3, 2025 •
edited
Loading

🦋 Changeset detected

Latest commit: 37fe3a8

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 21 packages
Name Type
@siafoundation/sdk Minor
@siafoundation/types Minor
@siafoundation/clusterd Patch
@siafoundation/design-system Patch
@siafoundation/explored-types Patch
@siafoundation/hostd-types Patch
@siafoundation/react-core Major
@siafoundation/renterd-types Patch
@siafoundation/units Patch
@siafoundation/walletd-mock Patch
@siafoundation/walletd-types Patch
@siafoundation/explored-js Patch
@siafoundation/explored-react Patch
@siafoundation/hostd-js Patch
@siafoundation/hostd-react Patch
@siafoundation/e2e Patch
@siafoundation/renterd-react Patch
@siafoundation/sia-central-react Patch
@siafoundation/walletd-react Patch
@siafoundation/renterd-js Patch
@siafoundation/walletd-js Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@alexfreska Graphite App
Copy link
Member Author

alexfreska commented Feb 3, 2025 •
edited
Loading

This stack of pull requests is managed by Graphite. Learn more about stacking.

This was referenced Feb 3, 2025
Merged
Merged
Merged
Merged
@alexfreska alexfreska force-pushed the feat_cluster_v2_transition branch from 1fd9da9 to 53c8c33 Compare February 3, 2025 17:16
@alexfreska alexfreska force-pushed the refactor_sdk_cleanup branch from 84b8b33 to f4f94d9 Compare February 3, 2025 17:16
@alexfreska alexfreska force-pushed the feat_cluster_v2_transition branch from 53c8c33 to 752178c Compare February 3, 2025 19:29
@alexfreska alexfreska force-pushed the refactor_sdk_cleanup branch from f4f94d9 to ba633a4 Compare February 3, 2025 19:29
@alexfreska alexfreska mentioned this pull request Feb 3, 2025
Merged
@alexfreska alexfreska force-pushed the refactor_sdk_cleanup branch from ba633a4 to 46d7c0c Compare February 3, 2025 19:35
@alexfreska alexfreska force-pushed the feat_cluster_v2_transition branch from 752178c to 5a36de1 Compare February 3, 2025 20:11
@alexfreska alexfreska force-pushed the refactor_sdk_cleanup branch from 46d7c0c to fd3b7eb Compare February 3, 2025 20:11
@alexfreska alexfreska mentioned this pull request Feb 3, 2025
Merged
@alexfreska alexfreska force-pushed the feat_cluster_v2_transition branch from 5a36de1 to 472bae5 Compare February 4, 2025 14:49
@alexfreska alexfreska force-pushed the refactor_sdk_cleanup branch from fd3b7eb to c79a9d2 Compare February 4, 2025 14:49
@alexfreska alexfreska force-pushed the feat_cluster_v2_transition branch from 472bae5 to 5c57d67 Compare February 4, 2025 15:09
@alexfreska alexfreska force-pushed the refactor_sdk_cleanup branch from c79a9d2 to ed7210e Compare February 4, 2025 15:10
@alexfreska alexfreska force-pushed the feat_cluster_v2_transition branch from 5c57d67 to 85cdc3d Compare February 4, 2025 17:46
@alexfreska alexfreska force-pushed the refactor_sdk_cleanup branch from ed7210e to a193de9 Compare February 4, 2025 17:47
@alexfreska alexfreska force-pushed the feat_cluster_v2_transition branch 2 times, most recently from 546ee22 to 781e1bc Compare February 4, 2025 19:58
@alexfreska alexfreska force-pushed the refactor_sdk_cleanup branch 2 times, most recently from 00c6291 to 1ddbc69 Compare February 4, 2025 22:43
@alexfreska Graphite App
Copy link
Member Author

alexfreska commented Feb 5, 2025 •
edited
Loading

Merge activity

  • Feb 5, 9:01 AM EST: A user started a stack merge that includes this pull request via Graphite.
  • Feb 5, 9:04 AM EST: Graphite rebased this pull request as part of a merge.
  • Feb 5, 9:05 AM EST: A user merged this pull request with Graphite.

@alexfreska alexfreska changed the base branch from feat_cluster_v2_transition to graphite-base/909 February 5, 2025 14:02
@alexfreska alexfreska changed the base branch from graphite-base/909 to main February 5, 2025 14:02
@alexfreska alexfreska force-pushed the refactor_sdk_cleanup branch from 1ddbc69 to 37fe3a8 Compare February 5, 2025 14:03
@alexfreska alexfreska merged commit ef647e5 into main Feb 5, 2025
16 of 33 checks passed
@alexfreska alexfreska deleted the refactor_sdk_cleanup branch February 5, 2025 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@peterjan peterjan peterjan approved these changes

@telestrial telestrial Awaiting requested review from telestrial

Assignees

@alexfreska alexfreska

Labels
None yet
Projects
Sia
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@alexfreska @peterjan