Bitcoin-Core-App icon indicating copy to clipboard operation
Bitcoin-Core-App copied to clipboard

Add a screen to onboarding for enabling notification permissions

Open GBKS opened this issue 2 years ago • 3 comments

Currently, when starting the app for the first time, the user is instantly greeted with a request to enable notifications. At this point, they are less likely to be able to make an informed decision since they have not gone through the first-use explainers, and the request modal does not come with a rationale. We can move this modal to a later point in the flow Here's a mock-up for a screen.

image

Tapping "Enable" would show the OS modal to enable the notification permission. Tapping "Skip" would simply go to the next screen.

A question I have is whether we then also need a new option in settings for this? Whether an app can send notifications or not is usually handled in the OS settings, not inside the app itself. But let's say a user enables this later, will the app know about this and can then create the system notification?

Feedback appreciated, as always.

GBKS avatar May 08 '23 06:05 GBKS

cc @johnny9

Can we implement this now? And can we then disconnect or reconnect the notification server through a settings menu option post-onboarding?

jarolrod avatar May 25 '23 19:05 jarolrod

Yes it is possible. It somewhat depends on https://github.com/bitcoin-core/gui-qml/pull/331 since the jni bridge will be used to implement it.

johnny9 avatar May 25 '23 20:05 johnny9

This is now documented in the design docs first use page (very bottom).

GBKS avatar Feb 08 '24 13:02 GBKS