woocommerce / woocommerce-android

WooCommerce Android app
https://www.woocommerce.com/mobile
GNU General Public License v2.0
277 stars 135 forks source link

Hide Jetpack upsells when site is black-flagged #12788

Closed hichamboushaba closed 1 month ago

hichamboushaba commented 1 month ago

Closes: #12782

Depends on https://github.com/woocommerce/woocommerce-android/pull/12780

Description

This PR adds the following changes:

Steps to reproduce

  1. DM me to share with a suspended website (or the steps to create one)
  2. Open the app and sign in to the website.

Testing information

The tests that have been performed

^

Reviewer (or Author, in the case of optional code reviews):

Please make sure these conditions are met before approving the PR, or request changes if the PR needs improvement:

dangermattic commented 1 month ago
2 Warnings
:warning: View files have been modified, but no screenshot or video is included in the pull request. Consider adding some for clarity.
:warning: This PR is assigned to the milestone 20.8. The due date for this milestone has already passed.
Please assign it to a milestone with a later deadline or check whether the release for this milestone has already been finished.

Generated by :no_entry_sign: Danger

wpmobilebot commented 1 month ago
📲 You can test the changes from this Pull Request in WooCommerce-Wear Android by scanning the QR code below to install the corresponding build.
App Name WooCommerce-Wear Android
Platform⌚️ Wear OS
FlavorJalapeno
Build TypeDebug
Commitc0d7adb930c584ec69e2788d9f668e824972d106
Direct Downloadwoocommerce-wear-prototype-build-pr12788-c0d7adb.apk
wpmobilebot commented 1 month ago

📲 You can test the changes from this Pull Request in WooCommerce Android by scanning the QR code below to install the corresponding build.

App Name WooCommerce Android
Platform📱 Mobile
FlavorJalapeno
Build TypeDebug
Commitc0d7adb930c584ec69e2788d9f668e824972d106
Direct Downloadwoocommerce-prototype-build-pr12788-c0d7adb.apk
codecov-commenter commented 1 month ago

Codecov Report

Attention: Patch coverage is 45.45455% with 6 lines in your changes missing coverage. Please review.

Project coverage is 40.80%. Comparing base (84c967a) to head (c0d7adb). Report is 16 commits behind head on trunk.

Files with missing lines Patch % Lines
...rc/main/kotlin/com/woocommerce/android/AppPrefs.kt 0.00% 4 Missing :warning:
.../kotlin/com/woocommerce/android/AppPrefsWrapper.kt 0.00% 1 Missing :warning:
...commerce/android/ui/prefs/MainSettingsPresenter.kt 50.00% 0 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## trunk #12788 +/- ## ========================================= Coverage 40.80% 40.80% - Complexity 5750 5753 +3 ========================================= Files 1237 1237 Lines 69827 69833 +6 Branches 9698 9699 +1 ========================================= + Hits 28493 28498 +5 + Misses 38707 38706 -1 - Partials 2627 2629 +2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.