Open Bug 1942663 Opened 1 year ago Updated 1 year ago

Revert the change made to canShowNotificationPage in 1942326

Categories

(Firefox for Android :: Onboarding, task, P3)

Product:
Firefox for Android
Component:
Onboarding
Platform:
All
Android
Type:
task

Tracking

()

People

(Reporter: towhite, Unassigned)

References

Details

(Whiteboard: [fxdroid][group2])

The areNotificationsEnabledSafe check was removed to fix a bug when used in combination with the Theme onboarding card. Removing the condition introduced a minor regression in a specific edge case. If a user has already enabled notifications (prior to launching onboarding via the app settings), the card will still be displayed. However, this behavior is consistent with the current experience, where the card remains visible even when notifications are enabled through onboarding.

This could potentially be further improved or reverted altogether once the theme card onboarding work is completed.

You need to log in before you can comment on or make changes to this bug.