Skip to content

Conversation

@flopez7
Copy link
Contributor

@flopez7 flopez7 commented Aug 28, 2025

Issue tracking

#3523

Context behind the change

  • Implement get method in KVStoreClient with error handling and add corresponding tests
  • Remove unused mocks and move static mocks at the top of the file

How has this been tested?

  • Ran tests
  • Made some queries locally

Release plan

Deploy new SDK version

Potential risks; What to monitor; Rollback plan

None

…orresponding tests

- Remove unused mocks and move static mocks at the top of the file
@vercel
Copy link

vercel bot commented Aug 28, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
human-app Ready Ready Preview Comment Aug 28, 2025 0:27am
human-dashboard-frontend Ready Ready Preview Comment Aug 28, 2025 0:27am
staking-dashboard Ready Ready Preview Comment Aug 28, 2025 0:27am
2 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
faucet-frontend Ignored Ignored Aug 28, 2025 0:27am
faucet-server Ignored Ignored Aug 28, 2025 0:27am

@flopez7 flopez7 changed the title [SDK] Add ability to read KVStore data using KVStoreClient [SDK] Read KVStore data using KVStoreClient Aug 28, 2025
@flopez7 flopez7 requested a review from portuu3 August 28, 2025 12:29
@flopez7 flopez7 self-assigned this Aug 28, 2025
@portuu3 portuu3 merged commit 354a940 into develop Sep 5, 2025
22 checks passed
@portuu3 portuu3 deleted the feat/sdk/kvstore-get branch September 5, 2025 11:14
@portuu3 portuu3 mentioned this pull request Sep 5, 2025
4 tasks
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.

3 participants