This repository was archived by the owner on Sep 8, 2025. It is now read-only.
fix: allow failed address checker users to perform unbond#1542
Open
jrwbabylonlab wants to merge 2 commits into
Open
fix: allow failed address checker users to perform unbond#1542jrwbabylonlab wants to merge 2 commits into
jrwbabylonlab wants to merge 2 commits into
Conversation
cc8a496 to
65ab8f1
Compare
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR implements address screening functionality for BTC addresses to prevent users with failed risk assessments from staking while allowing them to perform unbond operations. The key changes include adding a new address screening service, integrating it with the BTC wallet provider, and updating UI components to handle the disabled state appropriately.
- Address screening service with local storage caching
- Integration of risk assessment checks into BTC wallet connection flow
- UI updates to disable staking forms and expansion sections for flagged addresses
Reviewed Changes
Copilot reviewed 13 out of 13 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| src/ui/common/utils/local_storage/addressScreeningStorage.ts | New utility for caching BTC address screening results in localStorage |
| src/ui/common/hooks/services/useAddressScreeningService.ts | New service hook for address screening with caching logic |
| src/ui/common/context/wallet/BTCWalletProvider.tsx | Integration of address screening into wallet connection flow |
| src/ui/common/state/StakingState.tsx | Updated staking state to handle failed risk assessments |
| src/ui/common/state/StakingExpansionState.tsx | Added disabled state for expansion functionality |
| src/ui/common/components/Multistaking/MultistakingForm/SubmitButton.tsx | Updated submit button to handle disabled state |
| src/ui/common/components/Multistaking/MultistakingForm/FormAlert.tsx | Simplified alert component to accept title/message props |
| src/ui/common/components/Activity/components/StakeExpansionSection.tsx | Added disabled prop to accordion component |
| src/ui/baby/state/StakingState.tsx | Added disabled state handling for baby UI |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
5876c77 to
dcb8900
Compare
jeremy-babylonlabs
approved these changes
Sep 5, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
expansion is disabled

What's missing:
Show messages when mouse hover on top of the expansion section