Skip to content

Various fixes#199

Merged
thewhaleking merged 11 commits intostagingfrom
fix/thewhaleking/various-fixes
Nov 1, 2024
Merged

Various fixes#199
thewhaleking merged 11 commits intostagingfrom
fix/thewhaleking/various-fixes

Conversation

@thewhaleking
Copy link
Contributor

@thewhaleking thewhaleking commented Oct 30, 2024

  • in do_transfer, we check the balance with coldkeypub.ss58, but then retrieve it from the dict with coldkey.ss58. Resolve this.
  • Handle KeyboardInterrupt in CLI to gracefully exit (no traceback)
  • Handle race conditions where self.metadata may not be set before finishing initialising runtime (this may need optimised in the future)
  • Error description output
  • Taostats link fixed
  • Fixes not showing confirmation if --no-prompt is specified on stake remove

Fixes #202

@thewhaleking thewhaleking changed the base branch from main to staging October 30, 2024 15:20
@thewhaleking thewhaleking marked this pull request as ready for review October 31, 2024 15:25
@thewhaleking thewhaleking requested review from a team and ibraheem-abe November 1, 2024 09:52
@thewhaleking thewhaleking merged commit 1bb0ec7 into staging Nov 1, 2024
@thewhaleking thewhaleking deleted the fix/thewhaleking/various-fixes branch November 1, 2024 15:38
@ibraheem-abe ibraheem-abe mentioned this pull request Nov 5, 2024
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