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

[SDK] Change KVStoreKeys.name to KVStoreKeys.operatorName #3075

Merged
merged 1 commit into from
Feb 10, 2025

Conversation

portuu3
Copy link
Collaborator

@portuu3 portuu3 commented Feb 6, 2025

Issue tracking

NA

Context behind the change

In python autogenerated docs name is a reserved word and was not being included in the docs generated.
It was decided to rename it to avoid this issue.

How has this been tested?

NA

Release plan

NA

Potential risks; What to monitor; Rollback plan

NA

@portuu3 portuu3 self-assigned this Feb 6, 2025
Copy link

vercel bot commented Feb 6, 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 6, 2025 0:13am
human-dashboard-frontend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 6, 2025 0:13am
staking-dashboard ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 6, 2025 0:13am
2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
faucet-frontend ⬜️ Ignored (Inspect) Visit Preview Feb 6, 2025 0:13am
faucet-server ⬜️ Ignored (Inspect) Visit Preview Feb 6, 2025 0:13am

@portuu3 portuu3 merged commit 34d5e5a into develop Feb 10, 2025
23 checks passed
@portuu3 portuu3 deleted the kvstore-name-key branch February 10, 2025 09:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants