-
Notifications
You must be signed in to change notification settings - Fork 55
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
bugfix: fixed parsing account id for fetching active staking pools #369
Conversation
@akorchyn Thank you for your contribution! Your pull request is now a part of the Race of Sloths! Current status: executed
The average score is 3 @akorchyn check out your results on the Race of Sloths Leaderboard! and in the profile What is the Race of SlothsRace of Sloths is a friendly competition where you can participate in challenges and compete with other open-source contributors within your normal workflow For contributors:
For maintainers:
Feel free to check our website for additional details! Bot commands
|
@race-of-sloths score 3 |
🌟 Score recorded!@frol, thank you for scoring this pull request in the Race of Sloths! |
✅ PR is finalized!Your contribution is much appreciated with a final score of 3! Another weekly streak completed, well done @akorchyn! To keep your weekly streak and get another bonus make pull request next week! Looking forward to see you in race-of-sloths |
## 🤖 New release * `near-cli-rs`: 0.12.0 -> 0.13.0 <details><summary><i><b>Changelog</b></i></summary><p> <blockquote> ## [0.13.0](v0.12.0...v0.13.0) - 2024-07-30 ### Added - Automatically exec legacy JS CLI commands for full backward compatibility ([#366](#366)) - Added the ability to use the TEACH-ME mode ([#360](#360)) - Added a new subcommand to edit configuration parameters ([#367](#367)) ### Fixed - Fixed the fallback implementation of fetching active staking pools ([#369](#369)) ### Other - Fixed typos in user prompts and the guide ([#372](#372)) </blockquote> </p></details> --- This PR was generated with [release-plz](https://github.com/MarcoIeni/release-plz/).
What was wrong
The provided string contained a data prefix before the account id name that prevented the function from parsing the AccountId
Also, I extracted the filter into the call itself
See image:
@race-of-sloths