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

Convert roles from human readable to snake case to unify values #3054

Merged
merged 4 commits into from
Feb 6, 2025

Conversation

flopez7
Copy link
Contributor

@flopez7 flopez7 commented Jan 29, 2025

Issue tracking

#3019

Context behind the change

Unify the KVStore values for the key role to be snake_case

How has this been tested?

  • Launched tests
  • Deployed locally dashboard, dashboard server and staking dasboard for some tests.

Release plan

  • Deploy new SDK version
  • Deploy dashboard server
  • Deploy dashboard client
  • Deploy staking dashboard
  • Update all oracles roles stored in KVStore

Potential risks; What to monitor; Check if roles are being displayed properly.

Copy link

vercel bot commented Jan 29, 2025

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 Feb 4, 2025 2:35pm
human-dashboard-frontend ❌ Failed (Inspect) Feb 4, 2025 2:35pm
staking-dashboard ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 4, 2025 2:35pm
2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
faucet-frontend ⬜️ Ignored (Inspect) Visit Preview Feb 4, 2025 2:35pm
faucet-server ⬜️ Ignored (Inspect) Visit Preview Feb 4, 2025 2:35pm

@portuu3 portuu3 merged commit 44f16df into develop Feb 6, 2025
20 of 21 checks passed
@portuu3 portuu3 deleted the feat/sdk/convert-role-snake-case branch February 6, 2025 14:00
@portuu3 portuu3 mentioned this pull request Feb 6, 2025
16 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.

2 participants