8000 Fix getStakingInfoFromAddressBook error handling by blukat29 · Pull Request #1882 · klaytn/klaytn · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
This repository was archived by the owner on Aug 19, 2024. It is now read-only.

Fix getStakingInfoFromAddressBook error handling #1882

Merged
merged 1 commit into from
Jul 18, 2023
Merged

Fix getStakingInfoFromAddressBook error handling #1882

merged 1 commit into from
Jul 18, 2023

Conversation

blukat29
Copy link
Contributor

Proposed changes

Make getStakingInfoFromAddressBook behave as similar as possible as was in v1.10.2.

Types of changes

  • Bugfix
  • New feature or enhancement
  • Others

Checklist

  • I have read the CONTRIBUTING GUIDELINES doc
  • I have signed the CLA
  • Lint and unit tests pass locally with my changes ($ make test)
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)
  • Any dependent changes have been merged and published in downstream modules

Related issues

Further comments

@blukat29 blukat29 requested a review from ian0371 as a code owner July 18, 2023 05:08
@blukat29 blukat29 requested review from aidan-kwon and 2dvorak July 18, 2023 05:08
@blukat29 blukat29 self-assigned this Jul 18, 2023
@blukat29 blukat29 merged commit 80ae27a into klaytn:dev Jul 18, 2023
@blukat29 blukat29 deleted the addressbook-caller-errors branch July 18, 2023 09:05
@2dvorak 2dvorak mentioned this pull request Jul 25, 2023
Merged
9 tasks
@aidan-kwon aidan-kwon mentioned this pull request Aug 3, 2023
Closed
20 tasks
< 79ED input type="hidden" data-csrf="true" name="authenticity_token" value="HUSGXjpd9GY0qDQXLVE0fldhbNASPPhwV4h0RmRGmf2MHxs7486DMAwv4CPqWMDws99wAI20Axfsvs4RxwdDbg==" />
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ian0371 ian0371 ian0371 approved these changes

@hyunsooda hyunsooda hyunsooda approved these changes

@aidan-kwon aidan-kwon aidan-kwon approved these changes

@2dvorak 2dvorak Awaiting requested review from 2dvorak

Assignees

@blukat29 blukat29

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@blukat29 @ian0371 @hyunsooda @aidan-kwon
0