Skip to content
This repository was archived by the owner on Sep 8, 2025. It is now read-only.

Conversation

@jrwbabylonlab
Copy link
Collaborator

@jrwbabylonlab jrwbabylonlab commented Sep 4, 2025

image

expansion is disabled
image

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

@jrwbabylonlab jrwbabylonlab force-pushed the allow-failed-address-checker-unbond branch 2 times, most recently from cc8a496 to 65ab8f1 Compare September 5, 2025 06:42
@jrwbabylonlab jrwbabylonlab marked this pull request as ready for review September 5, 2025 07:40
Copilot AI review requested due to automatic review settings September 5, 2025 07:40
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@jeremy-babylonlabs jeremy-babylonlabs force-pushed the allow-failed-address-checker-unbond branch from 5876c77 to dcb8900 Compare September 5, 2025 12:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants