Add kvstore to Staking dashboard #3036

flopez7 · 2025-01-22T14:18:05Z

Issue tracking

#2973

Context behind the change

Added new hook for getting data from subgraph and update it onchain
Added a table for showing KVStore data
Added modal for editing KVStore data

How has this been tested?

Launched locally

Release plan

Deploy Dashboard server
Deploy Staking Dashboard

Potential risks; What to monitor; None

vercel · 2025-01-22T14:18:11Z

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

Name	Status	Preview	Comments	Updated (UTC)
human-app	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 27, 2025 2:38pm
human-dashboard-frontend	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 27, 2025 2:38pm
staking-dashboard	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 27, 2025 2:38pm

2 Skipped Deployments

Name	Status	Preview	Comments	Updated (UTC)
faucet-frontend	⬜️ Ignored (Inspect)	Visit Preview		Jan 27, 2025 2:38pm
faucet-server	⬜️ Ignored (Inspect)	Visit Preview		Jan 27, 2025 2:38pm

portuu3 · 2025-01-22T15:46:12Z

Make the modal larger if the screen is big enough:
I can click on save changes without any change and wallet asks for confirmation.
I have just added a new key (name), and sets again others that have not been updated.
https://amoy.polygonscan.com/tx/0x42e9a69da70316143333fabe8029855f39f226878f6eb1307a2884e9f6f00f14#eventlog
Category should have a dropdown with the 2 possible options.
I have changed from Amoy to BSC testnet and network label is not visible. Also after this network change I have tried to set a new key and the call has been sent to kvstore address from Amoy in BSC testnet. This transactions should have failed since the contract is not at this address.
https://testnet.bscscan.com/tx/0x6ede4cbbb5839bcb1e1552c96d331d3df89c735f979673b44e6b71a85475805f
Update: it was related to VITE_APP_SUPPORTED_CHAINS not including BSC testnet, but it should not allow to send the transaction in that case and should not load kvstore data.

packages/apps/dashboard/server/src/modules/details/details.service.ts

Test for dashboard server method Check chainId in KVStore methods Make sure there's no empty keys on set/setup requests

flopez7 · 2025-01-23T15:54:21Z

Solved @portuu3

Modify hooks to remove invalid chain error when wallet is not connected

Remove / from VITE_APP_DASHBOARD_API_URL

portuu3 · 2025-01-27T07:42:44Z

It keeps setting the same value it has to all keys when the you update only one (in this case I updated url)
https://amoy.polygonscan.com/tx/0x2c9c6684f33ea1996a3714df5168b8646a9c0b55ea7ca84ef94beb5db9c92a83#eventlog

portuu3 · 2025-01-27T07:44:18Z

public_key_hash should be one of the dropdown keys

packages/sdk/python/human-protocol-sdk/human_protocol_sdk/constants.py

Add kvstore to Staking dashboard

2b787eb

flopez7 self-assigned this Jan 22, 2025

vercel bot deployed to Preview – staking-dashboard January 22, 2025 14:19 View deployment

portuu3 requested changes Jan 22, 2025

View reviewed changes

packages/apps/dashboard/server/src/modules/details/details.service.ts Show resolved Hide resolved

Add category selector

86da9d4

Test for dashboard server method Check chainId in KVStore methods Make sure there's no empty keys on set/setup requests

vercel bot deployed to Preview – human-app January 23, 2025 14:54 View deployment

vercel bot deployed to Preview – staking-dashboard January 23, 2025 14:56 View deployment

vercel bot deployed to Preview – human-dashboard-frontend January 23, 2025 14:57 View deployment

Merge branch 'develop' into feat/staking/kvstore

f5bd2ac

vercel bot deployed to Preview – staking-dashboard January 23, 2025 16:14 View deployment

vercel bot deployed to Preview – faucet-server January 23, 2025 16:19 View deployment

vercel bot deployed to Preview – human-dashboard-frontend January 23, 2025 16:23 View deployment

vercel bot deployed to Preview – human-app January 23, 2025 16:27 View deployment

vercel bot deployed to Preview – faucet-frontend January 23, 2025 16:31 View deployment

Remove node-sass from ui-2024

800e378

Modify hooks to remove invalid chain error when wallet is not connected

vercel bot deployed to Preview – staking-dashboard January 24, 2025 09:50 View deployment

vercel bot deployed to Preview – human-dashboard-frontend January 24, 2025 09:54 View deployment

vercel bot deployed to Preview – human-app January 24, 2025 09:59 View deployment

Remove filter for empty values in KVStore table

3a8349e

Remove / from VITE_APP_DASHBOARD_API_URL

vercel bot deployed to Preview – human-app January 24, 2025 10:40 View deployment

vercel bot deployed to Preview – staking-dashboard January 24, 2025 10:42 View deployment

vercel bot deployed to Preview – human-dashboard-frontend January 24, 2025 10:43 View deployment

Improved kvstore hook by using api service and added response validation

9b47fba

vercel bot deployed to Preview – human-app January 24, 2025 12:21 View deployment

vercel bot deployed to Preview – staking-dashboard January 24, 2025 12:23 View deployment

vercel bot deployed to Preview – human-dashboard-frontend January 24, 2025 12:25 View deployment

Enable again kvstore filter

57d4b75

vercel bot deployed to Preview – staking-dashboard January 24, 2025 12:52 View deployment

vercel bot deployed to Preview – human-app January 24, 2025 12:53 View deployment

vercel bot deployed to Preview – human-dashboard-frontend January 24, 2025 12:54 View deployment

Modify input type set number for fee

20eb5bf

vercel bot deployed to Preview – staking-dashboard January 24, 2025 13:51 View deployment

vercel bot deployed to Preview – human-app January 24, 2025 13:53 View deployment

vercel bot deployed to Preview – human-dashboard-frontend January 24, 2025 13:54 View deployment

Add validation to exclude custom keys that are not updated

ebf9cb5

vercel bot deployed to Preview – human-dashboard-frontend January 27, 2025 10:19 View deployment

vercel bot deployed to Preview – human-app January 27, 2025 10:20 View deployment

vercel bot deployed to Preview – staking-dashboard January 27, 2025 10:22 View deployment

portuu3 requested changes Jan 27, 2025

View reviewed changes

packages/sdk/python/human-protocol-sdk/human_protocol_sdk/constants.py Outdated Show resolved Hide resolved

Fix prettier typing error

f34760f

vercel bot deployed to Preview – human-app January 27, 2025 14:35 View deployment

vercel bot deployed to Preview – staking-dashboard January 27, 2025 14:37 View deployment

vercel bot deployed to Preview – human-dashboard-frontend January 27, 2025 14:38 View deployment

portuu3 approved these changes Jan 27, 2025

View reviewed changes

portuu3 merged commit 9953bfa into develop Jan 27, 2025
23 checks passed

portuu3 deleted the feat/staking/kvstore branch January 27, 2025 15:18

flopez7 mentioned this pull request Jan 28, 2025

Release 2025-02-05 #3017

Merged

35 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add kvstore to Staking dashboard #3036

Add kvstore to Staking dashboard #3036

flopez7 commented Jan 22, 2025

vercel bot commented Jan 22, 2025 •

edited

Loading

portuu3 commented Jan 22, 2025 •

edited

Loading

flopez7 commented Jan 23, 2025

portuu3 commented Jan 27, 2025

portuu3 commented Jan 27, 2025

Add kvstore to Staking dashboard #3036

Add kvstore to Staking dashboard #3036

Conversation

flopez7 commented Jan 22, 2025

Issue tracking

Context behind the change

How has this been tested?

Release plan

Potential risks; What to monitor; None

vercel bot commented Jan 22, 2025 • edited Loading

portuu3 commented Jan 22, 2025 • edited Loading

flopez7 commented Jan 23, 2025

portuu3 commented Jan 27, 2025

portuu3 commented Jan 27, 2025

vercel bot commented Jan 22, 2025 •

edited

Loading

portuu3 commented Jan 22, 2025 •

edited

Loading