Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Optimize the logic of "no blaze campaign" local notification #12585

Merged
merged 7 commits into from
Sep 12, 2024
Merged

Optimize the logic of "no blaze campaign" local notification #12585

merged 7 commits into from
Sep 12, 2024

Commits on Sep 11, 2024

  1. Add blazeFirstTimeWithoutCampaign pref to AppPrefs

    @irfano
    irfano committed Sep 11, 2024
    Configuration menu
    Copy the full SHA
    2bf764e View commit details
    Browse the repository at this point in the history

  2. Improve the logic in BlazeCampaignsObserver

    @irfano
    irfano committed Sep 11, 2024
    Configuration menu
    Copy the full SHA
    1e1035b View commit details
    Browse the repository at this point in the history

  3. Remove blazeFirstTimeWithoutCampaign key when notification is shown.

    @irfano
    irfano committed Sep 11, 2024
    Configuration menu
    Copy the full SHA
    3caf81e View commit details
    Browse the repository at this point in the history

  4. Add debug logs for scheduling and canceling local notifications

    @irfano
    irfano committed Sep 11, 2024
    Configuration menu
    Copy the full SHA
    cb029a9 View commit details
    Browse the repository at this point in the history

  5. Add more tests to BlazeCampaignsObserverTest

    @irfano
    irfano committed Sep 11, 2024
    Configuration menu
    Copy the full SHA
    b5ddc82 View commit details
    Browse the repository at this point in the history

Commits on Sep 12, 2024

  1. Merge branch 'trunk' into issue/12574-optimize-blaze-no-campaign-notf… 

    …ication
    @irfano
    irfano authored Sep 12, 2024
    Configuration menu
    Copy the full SHA
    08d8b96 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'refs/heads/trunk' into issue/12574-optimize-blaze-no-ca… 

    …mpaign-notfication

# Conflicts:
#	WooCommerce/src/main/kotlin/com/woocommerce/android/AppPrefs.kt
#	WooCommerce/src/main/kotlin/com/woocommerce/android/AppPrefsWrapper.kt
    @irfano
    irfano committed Sep 12, 2024
    Configuration menu
    Copy the full SHA
    9dd620f View commit details
    Browse the repository at this point in the history