Skip to content

Conversation

@shonsirsha
Copy link
Collaborator

@shonsirsha shonsirsha commented Dec 1, 2025

On iOS, Bluetooth can be “off” for two different reasons:

  1. System Bluetooth is turned off

  2. Bluetooth permission is disabled specifically for BitBoxApp

This commit adds logic to distinguish between these cases and
shows the appropriate message for each.

If system bluetooth is turned off:
Screenshot 2025-12-02 at 06 29 28

If bluetooth permission is disabled specifically for BBApp:
Screenshot 2025-12-01 at 14 01 52

On clicking "Enable", user will be brought to the Application settings:
IMG_6736

@shonsirsha shonsirsha force-pushed the frontend-improve-alert-message branch 3 times, most recently from 168ad88 to 726901c Compare December 1, 2025 13:46
@shonsirsha shonsirsha requested a review from benma December 2, 2025 05:11
@shonsirsha shonsirsha marked this pull request as ready for review December 2, 2025 05:12
@shonsirsha shonsirsha changed the title WIP frontend: improve BT alert message WIP frontend: improve BT alert message WIP Dec 2, 2025
@shonsirsha shonsirsha changed the title frontend: improve BT alert message WIP frontend: improve BT alert message Dec 2, 2025
@shonsirsha shonsirsha force-pushed the frontend-improve-alert-message branch 2 times, most recently from 3b9c913 to f426d75 Compare December 2, 2025 05:29
On iOS, Bluetooth can be “off” for two different reasons:

1. System Bluetooth is turned off

2. Bluetooth permission is disabled specifically for BitBoxApp

This commit adds logic to distinguish between these cases and
shows the appropriate message for each.
@shonsirsha shonsirsha force-pushed the frontend-improve-alert-message branch from f426d75 to c5fb98a Compare December 2, 2025 05:33
Copy link
Contributor

@benma benma left a comment

Choose a reason for hiding this comment

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

Looks nice, but I still need to test.

"title": "Welcome"
}
}
}
Copy link
Contributor

Choose a reason for hiding this comment

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

make locize-fix should add a newline at the end. If not, update locize cli, there was a change in some recent version.

{t('bluetooth.disabledPermissionDescription')}
</span>
<A className={styles.link} href="app-settings:">
Enable
Copy link
Contributor

Choose a reason for hiding this comment

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

localize this? maybe there is already a key that has this.

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