[Dynamic Dashboard] Orders card #11486

0nko · 2024-05-10T12:08:58Z

Implements #11461 and displays the Most recent orders dashboard card (without the filtering header).

To test:

Start the app
Notice a new Most recent orders card is shown
Verify the 3 most recent orders are displayed
Tap on View all orders
Verify the Orders tab is shown
Go back
Pull to refresh
Notice the card data is reloaded

dangermattic · 2024-05-10T12:09:29Z

	5 Warnings
⚠️	This PR is larger than 300 lines of changes. Please consider splitting it into smaller PRs for easier and faster reviews.
⚠️	Class `DashboardOrdersViewModel` is missing tests, but `unit-tests-exemption` label was set to ignore this.
⚠️	Class `ViewState` is missing tests, but `unit-tests-exemption` label was set to ignore this.
⚠️	Class `Error` is missing tests, but `unit-tests-exemption` label was set to ignore this.
⚠️	This PR is assigned to the milestone 18.7. This milestone is due in less than 2 days. Please make sure to get it merged by then or assign it to a milestone with a later deadline.

Generated by 🚫 Danger

wpmobilebot · 2024-05-10T12:24:50Z

Found 1 violations:

The PR caused the following dependency changes:

expand

-+--- org.wordpress:fluxc:trunk-cfe34975d20df76a2b7b4522dccd2faf53ef0f7c
-|    +--- org.wordpress:wellsql:2.0.0
-|    |    +--- androidx.annotation:annotation:1.2.0 -> 1.7.0 (*)
-|    |    \--- org.wordpress.wellsql:wellsql-annotations:2.0.0
-|    +--- org.wordpress.fluxc:fluxc-annotations:trunk-cfe34975d20df76a2b7b4522dccd2faf53ef0f7c
-|    +--- org.greenrobot:eventbus:3.3.1 (*)
-|    +--- com.squareup.okhttp3:okhttp:4.9.0 -> 4.12.0 (*)
-|    +--- com.android.volley:volley:1.1.1 -> 1.2.0
-|    +--- androidx.paging:paging-runtime:2.1.2
-|    |    +--- androidx.paging:paging-common:2.1.2
-|    |    |    +--- androidx.annotation:annotation:1.0.0 -> 1.7.0 (*)
-|    |    |    \--- androidx.arch.core:core-common:2.0.0 -> 2.2.0 (*)
-|    |    +--- androidx.arch.core:core-runtime:2.0.0 -> 2.2.0 (*)
-|    |    +--- androidx.lifecycle:lifecycle-runtime:2.0.0 -> 2.6.2 (*)
-|    |    +--- androidx.lifecycle:lifecycle-livedata:2.0.0 -> 2.6.2 (*)
-|    |    \--- androidx.recyclerview:recyclerview:1.0.0 -> 1.3.2 (*)
-|    +--- com.goterl:lazysodium-android:5.0.2
-|    +--- net.java.dev.jna:jna:5.5.0
-|    +--- org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.6.20 -> 1.9.10 (*)
-|    +--- org.jetbrains.kotlin:kotlin-android-extensions-runtime:1.6.20 -> 1.9.22
-|    |    \--- org.jetbrains.kotlin:kotlin-stdlib:1.9.22 (*)
-|    +--- androidx.appcompat:appcompat:1.0.2 -> 1.6.1 (*)
-|    +--- androidx.recyclerview:recyclerview:1.0.0 -> 1.3.2 (*)
-|    +--- androidx.exifinterface:exifinterface:1.0.0 -> 1.3.6
-|    |    \--- androidx.annotation:annotation:1.2.0 -> 1.7.0 (*)
-|    +--- androidx.security:security-crypto:1.0.0 -> 1.1.0-alpha03
-|    |    +--- androidx.annotation:annotation:1.1.0 -> 1.7.0 (*)
-|    |    +--- com.google.crypto.tink:tink-android:1.5.0
-|    |    \--- androidx.collection:collection:1.1.0 -> 1.4.0 (*)
-|    +--- com.squareup.okhttp3:okhttp-urlconnection:4.9.0
-|    |    +--- com.squareup.okhttp3:okhttp:4.9.0 -> 4.12.0 (*)
-|    |    \--- org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.4.10 -> 1.9.10 (*)
-|    +--- com.google.code.gson:gson:2.8.5 -> 2.10.1
-|    +--- org.apache.commons:commons-text:1.10.0 (*)
-|    +--- androidx.room:room-runtime:2.4.2 -> 2.5.2 (*)
-|    +--- androidx.room:room-ktx:2.4.2 -> 2.5.2
-|    |    +--- androidx.room:room-common:2.5.2 (*)
-|    |    +--- androidx.room:room-runtime:2.5.2 (*)
-|    |    +--- org.jetbrains.kotlin:kotlin-stdlib:1.7.20 -> 1.9.22 (*)
-|    |    \--- org.jetbrains.kotlinx:kotlinx-coroutines-android:1.6.4 -> 1.7.3 (*)
-|    +--- com.google.dagger:dagger:2.42 -> 2.50
-|    |    \--- javax.inject:javax.inject:1
-|    +--- org.jetbrains.kotlinx:kotlinx-coroutines-core:1.3.9 -> 1.7.3 (*)
-|    \--- org.jetbrains.kotlinx:kotlinx-coroutines-android:1.3.9 -> 1.7.3 (*)
++--- org.wordpress:fluxc:3008-f3ff9689a0a998537a624ce9a77b930a341bee48
+|    +--- org.wordpress:wellsql:2.0.0
+|    |    +--- androidx.annotation:annotation:1.2.0 -> 1.7.0 (*)
+|    |    \--- org.wordpress.wellsql:wellsql-annotations:2.0.0
+|    +--- org.wordpress.fluxc:fluxc-annotations:3008-f3ff9689a0a998537a624ce9a77b930a341bee48
+|    +--- org.greenrobot:eventbus:3.3.1 (*)
+|    +--- com.squareup.okhttp3:okhttp:4.9.0 -> 4.12.0 (*)
+|    +--- com.android.volley:volley:1.1.1 -> 1.2.0
+|    +--- androidx.paging:paging-runtime:2.1.2
+|    |    +--- androidx.paging:paging-common:2.1.2
+|    |    |    +--- androidx.annotation:annotation:1.0.0 -> 1.7.0 (*)
+|    |    |    \--- androidx.arch.core:core-common:2.0.0 -> 2.2.0 (*)
+|    |    +--- androidx.arch.core:core-runtime:2.0.0 -> 2.2.0 (*)
+|    |    +--- androidx.lifecycle:lifecycle-runtime:2.0.0 -> 2.6.2 (*)
+|    |    +--- androidx.lifecycle:lifecycle-livedata:2.0.0 -> 2.6.2 (*)
+|    |    \--- androidx.recyclerview:recyclerview:1.0.0 -> 1.3.2 (*)
+|    +--- com.goterl:lazysodium-android:5.0.2
+|    +--- net.java.dev.jna:jna:5.5.0
+|    +--- org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.6.20 -> 1.9.10 (*)
+|    +--- org.jetbrains.kotlin:kotlin-android-extensions-runtime:1.6.20 -> 1.9.22
+|    |    \--- org.jetbrains.kotlin:kotlin-stdlib:1.9.22 (*)
+|    +--- androidx.appcompat:appcompat:1.0.2 -> 1.6.1 (*)
+|    +--- androidx.recyclerview:recyclerview:1.0.0 -> 1.3.2 (*)
+|    +--- androidx.exifinterface:exifinterface:1.0.0 -> 1.3.6
+|    |    \--- androidx.annotation:annotation:1.2.0 -> 1.7.0 (*)
+|    +--- androidx.security:security-crypto:1.0.0 -> 1.1.0-alpha03
+|    |    +--- androidx.annotation:annotation:1.1.0 -> 1.7.0 (*)
+|    |    +--- com.google.crypto.tink:tink-android:1.5.0
+|    |    \--- androidx.collection:collection:1.1.0 -> 1.4.0 (*)
+|    +--- com.squareup.okhttp3:okhttp-urlconnection:4.9.0
+|    |    +--- com.squareup.okhttp3:okhttp:4.9.0 -> 4.12.0 (*)
+|    |    \--- org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.4.10 -> 1.9.10 (*)
+|    +--- com.google.code.gson:gson:2.8.5 -> 2.10.1
+|    +--- org.apache.commons:commons-text:1.10.0 (*)
+|    +--- androidx.room:room-runtime:2.4.2 -> 2.5.2 (*)
+|    +--- androidx.room:room-ktx:2.4.2 -> 2.5.2
+|    |    +--- androidx.room:room-common:2.5.2 (*)
+|    |    +--- androidx.room:room-runtime:2.5.2 (*)
+|    |    +--- org.jetbrains.kotlin:kotlin-stdlib:1.7.20 -> 1.9.22 (*)
+|    |    \--- org.jetbrains.kotlinx:kotlinx-coroutines-android:1.6.4 -> 1.7.3 (*)
+|    +--- com.google.dagger:dagger:2.42 -> 2.50
+|    |    \--- javax.inject:javax.inject:1
+|    +--- org.jetbrains.kotlinx:kotlinx-coroutines-core:1.3.9 -> 1.7.3 (*)
+|    \--- org.jetbrains.kotlinx:kotlinx-coroutines-android:1.3.9 -> 1.7.3 (*)
-\--- org.wordpress.fluxc.plugins:woocommerce:trunk-cfe34975d20df76a2b7b4522dccd2faf53ef0f7c
-     +--- org.wordpress:wellsql:2.0.0 (*)
-     +--- org.wordpress.fluxc:fluxc-annotations:trunk-cfe34975d20df76a2b7b4522dccd2faf53ef0f7c
-     +--- androidx.room:room-ktx:2.4.2 -> 2.5.2 (*)
-     +--- org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.6.20 -> 1.9.10 (*)
-     +--- org.wordpress:fluxc:trunk-cfe34975d20df76a2b7b4522dccd2faf53ef0f7c (*)
-     +--- com.google.code.gson:gson:2.8.5 -> 2.10.1
-     +--- com.google.dagger:dagger:2.42 -> 2.50 (*)
-     +--- org.jetbrains.kotlinx:kotlinx-coroutines-core:1.3.9 -> 1.7.3 (*)
-     +--- org.jetbrains.kotlinx:kotlinx-coroutines-android:1.3.9 -> 1.7.3 (*)
-     \--- androidx.room:room-runtime:2.4.2 -> 2.5.2 (*)
+\--- org.wordpress.fluxc.plugins:woocommerce:3008-f3ff9689a0a998537a624ce9a77b930a341bee48
+     +--- org.wordpress:wellsql:2.0.0 (*)
+     +--- org.wordpress.fluxc:fluxc-annotations:3008-f3ff9689a0a998537a624ce9a77b930a341bee48
+     +--- androidx.room:room-ktx:2.4.2 -> 2.5.2 (*)
+     +--- org.jetbrains.kotlin:kotlin-stdlib-jdk8:1.6.20 -> 1.9.10 (*)
+     +--- org.wordpress:fluxc:3008-f3ff9689a0a998537a624ce9a77b930a341bee48 (*)
+     +--- com.google.code.gson:gson:2.8.5 -> 2.10.1
+     +--- com.google.dagger:dagger:2.42 -> 2.50 (*)
+     +--- org.jetbrains.kotlinx:kotlinx-coroutines-core:1.3.9 -> 1.7.3 (*)
+     +--- org.jetbrains.kotlinx:kotlinx-coroutines-android:1.3.9 -> 1.7.3 (*)
+     \--- androidx.room:room-runtime:2.4.2 -> 2.5.2 (*)

Please review and act accordingly

wpmobilebot · 2024-05-10T12:40:10Z

📲 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
	Flavor	Jalapeno
	Build Type	Debug
	Commit	`b9d2611`
	Direct Download	`woocommerce-prototype-build-pr11486-b9d2611.apk`

hichamboushaba · 2024-05-10T13:24:45Z

@0nko before reviewing this, can you please rebase the PR on top of the branch of #11475, and use a logic similar to this commit (3594055) to make sure the card is now shown until M2 is ready?
If rebasing is not straightforward, then you can merge trunk into your branch after merging #11475.

0nko · 2024-05-10T14:40:05Z

@hichamboushaba done! Ready for a review.

hichamboushaba

Nice work @0nko 👏, I left some comments, mostly nit picks, but the first one is important IMO.

hichamboushaba · 2024-05-10T14:52:24Z

WooCommerce/src/main/kotlin/com/woocommerce/android/model/Order.kt

@@ -52,6 +52,7 @@ data class Order(
    val selectedGiftCard: String?,
    val giftCardDiscountedAmount: BigDecimal?,
    val shippingTax: BigDecimal,
+    val billingName: String = ""


I see some issues with this change to introduce billingName as a read-only property:

Adds to the size of the Order Parcelable, which is a minor thing, but something we should still avoid unless necessary.

Could lead to inconsistencies:

we can update the billingAddress using copy function, which I believe we do in the Order edit screen, so when this happen, the billingName will still point to an outdated value.

For someone unfamiliar with the code, they could think that updating the billingName is enough to update the customer's name, which is not the case.

As a solution, I suggest two solutions:

Either use the existing function Order#getBillingName

Introduce a computed property billingName that returns an empty string instead of the default value we use now, and then supply the default value in the UI code, this would be something like:

@IgnoredOnParcel val billingName get() = getBillingName("").trim()

WDYT?

Good point, thanks.

hichamboushaba · 2024-05-10T15:18:45Z

WooCommerce/src/main/kotlin/com/woocommerce/android/ui/dashboard/DashboardFragment.kt

@@ -165,6 +166,8 @@ class DashboardFragment :

                is FeedbackNegativeAction -> mainNavigationRouter?.showFeedbackSurvey()

+                is NavigateToOrders -> mainNavigationRouter?.showOrders()


np, seeing you introduce the function showOrders here got me thinking on why we need it, and it turns out, we don't, the Navigation Component and the integration with the Bottom Navigation can handle this for us with a simple call to findNavController.navigate(R.id.orders), this means that if you want, you can handle this navigation event directly from the card in a HandleEvent function, and decouple it from the parent ViewModel, but it's a minor thing, feel free to ignore it.

I thought there might be an easier way but I didn't think of this, thanks!

I followed your advice but now the bottom navigation button don't work right. I pushed the code so you could take a look while I'm trying to figure out if I can make it work, or if I need to revert the change.

I followed your advice but now the bottom navigation button don't work right. I pushed the code so you could take a look while I'm trying to figure out if I can make it work, or if I need to revert the change.

Sorry I missed this case when I suggested this solution, I found a fix after checking the code of NavigationUI.onNavDestinationSelected:

navController.navigateSafely( resId = R.id.orders, navOptions = navOptions { popUpTo(navController.graph.findStartDestination().id) { saveState = true } } )

But I'm not sure if this is simpler than what you had before or not, I'll leave it to you to decide which solution is better.

Nice, thank you!

hichamboushaba · 2024-05-10T15:21:46Z

WooCommerce/src/main/kotlin/com/woocommerce/android/ui/orders/list/OrderListRepository.kt

@@ -122,6 +126,38 @@ class OrderListRepository @Inject constructor(
        }
    }

+    fun observeTopOrders(count: Int, statusFilter: Order.Status? = null) = flow {
+        emit(Result.success(emptyList()))


np, do we need to start with an empty list? I think we can avoid the double emission by just removing the takeIf { it.isNotEmpty() } below, WDYT?

hichamboushaba · 2024-05-10T15:23:45Z

WooCommerce/src/main/kotlin/com/woocommerce/android/ui/dashboard/data/DashboardRepository.kt

@@ -88,10 +88,9 @@ class DashboardRepository @Inject constructor(
                isSelected = widget.isAdded,
                status = when (type) {
                    DashboardWidget.Type.STATS,
+                    DashboardWidget.Type.ORDERS,
                    DashboardWidget.Type.POPULAR_PRODUCTS -> statsWidgetsStatus


Nice that you used the hasOrders logic to hide the card 👏, just one np point, I think it would be more clearer to rename the class and property now, saying statsWidgetStatus is not very clear.

And also we should probably let the iOS team know about this decision (unless this was done and I missed it).

hichamboushaba · 2024-05-10T15:29:55Z

...erce/src/main/kotlin/com/woocommerce/android/ui/dashboard/orders/DashboardOrdersViewModel.kt

+                    if (orders.isEmpty()) {
+                        ViewState.Loading


np, coupling the empty state to loading could break easily, Do you think we can avoid it with using transformLatest? It's something I did in the reviews card and it worked well.

hichamboushaba · 2024-05-10T15:40:11Z

WooCommerce/src/main/kotlin/com/woocommerce/android/ui/orders/list/OrderListRepository.kt

+                orders
+                    .toList()
+                    .takeIf { it.isNotEmpty() }
+                    ?.let { emit(Result.success(it)) }


With this logic, we'll emit the cached value first even when the user pulls-to-refresh, WDYT about passing an argument isForced and skip the cache when it's true?

hichamboushaba · 2024-05-13T12:22:34Z

@0nko regarding the UI tests failure, it impacted my PR too, and it's because of a bug with the scrolling logic that I added on #11454, I attempted to improve the logic in this commit 8591359, feel free to cherry-pick it to your branch to fix the tests.

0nko · 2024-05-14T15:10:55Z

Thank you for the great feedback, @hichamboushaba!

0nko · 2024-05-14T21:03:02Z

@hichamboushaba I think I've addressed all feedback, ready for another round!

hichamboushaba

Nice work @0nko 🎖️, thanks for taking the time to implement my suggestions 🙏

As discussed, the UI tests are still failing, I managed to fix them on my PR #11491, so if it's OK for you, we can merge that PR first, then proceed to merging your after merging trunk, WDYT?

# Conflicts: # WooCommerce/src/main/kotlin/com/woocommerce/android/model/DashboardWidget.kt # WooCommerce/src/main/kotlin/com/woocommerce/android/ui/dashboard/DashboardContainer.kt # WooCommerce/src/main/kotlin/com/woocommerce/android/ui/dashboard/data/DashboardDataStore.kt # WooCommerce/src/main/res/values/strings.xml

codecov-commenter · 2024-05-15T21:11:59Z

Codecov Report

Attention: Patch coverage is 0.80321% with 247 lines in your changes are missing coverage. Please review.

Project coverage is 40.49%. Comparing base (c230ce5) to head (b9d2611).
Report is 5 commits behind head on trunk.

Files	Patch %	Lines
...android/ui/dashboard/orders/DashboardOrdersCard.kt	0.00%	128 Missing ⚠️
...id/ui/dashboard/orders/DashboardOrdersViewModel.kt	0.00%	75 Missing ⚠️
...erce/android/ui/orders/list/OrderListRepository.kt	0.00%	26 Missing ⚠️
...om/woocommerce/android/ui/compose/component/Tag.kt	0.00%	6 Missing ⚠️
...e/android/ui/dashboard/data/DashboardRepository.kt	0.00%	6 Missing ⚠️
...ommerce/android/ui/dashboard/DashboardContainer.kt	0.00%	3 Missing ⚠️
...main/kotlin/com/woocommerce/android/model/Order.kt	0.00%	1 Missing ⚠️
...ce/android/ui/dashboard/data/DashboardDataStore.kt	0.00%	1 Missing ⚠️
...ndroid/ui/dashboard/data/ObserveSiteOrdersState.kt	0.00%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##              trunk   #11486      +/-   ##
============================================
- Coverage     40.64%   40.49%   -0.16%     
  Complexity     5181     5181              
============================================
  Files          1073     1075       +2     
  Lines         62609    62844     +235     
  Branches       8565     8610      +45     
============================================
+ Hits          25446    25447       +1     
- Misses        34875    35109     +234     
  Partials       2288     2288

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

0nko added the feature: dashboard Related to home screen project label May 10, 2024

0nko added this to the 18.7 milestone May 10, 2024

0nko requested a review from hichamboushaba May 10, 2024 12:08

This was referenced May 10, 2024

Last 3 orders card #11461

Closed

[Dynamic Dashboard] Fetch orders wordpress-mobile/WordPress-FluxC-Android#3008

Merged

0nko marked this pull request as ready for review May 10, 2024 12:19

0nko added the unit-tests-exemption label May 10, 2024

hichamboushaba added the status: do not merge Dependent on another PR, ready for review but not ready for merge. label May 10, 2024

hichamboushaba self-assigned this May 10, 2024

0nko added 9 commits May 10, 2024 16:33

Add a dashboard orders card ViewModel

3c7ee5d

Add orders card UI

7e74cf2

Update the Order model to contain billing name

4d43a0a

Update the Tag component UI to match the design

706c379

Add a method to observe the orders

9a61c90

Add a way to navigate to orders tab

4978ae9

Add the missing type to dashboard repository

e78a466

Fix ktlint issues

0af6cda

Update FluxC hash

146964a

0nko force-pushed the issue/11461-orders-card branch from ddffdd6 to 146964a Compare May 10, 2024 14:35

0nko changed the base branch from trunk to dynamic-dashboard/feature-flag-m2 May 10, 2024 14:36

Hide new cards behind a feature flag

6deab33

Explicitly specify we don't want to delete old orders

6bfa2fd

hichamboushaba approved these changes May 10, 2024

View reviewed changes

Base automatically changed from dynamic-dashboard/feature-flag-m2 to trunk May 10, 2024 15:56

0nko added 6 commits May 13, 2024 14:36

Change the way billing name is retrieved

5d9d1b2

Simplify navigation to orders

b9a104a

Rename the widget status to be more general

c71448d

Fix tests

c8e8d22

Skip the cache if the loading is forced

739b89b

Fix the loading status emission

b3565d9

0nko requested a review from a team as a code owner May 14, 2024 15:08

0nko added 2 commits May 14, 2024 23:00

Fix the bottom navigation

408328a

Fix ktlint issues

f54a93c

hichamboushaba approved these changes May 15, 2024

View reviewed changes

0nko added 3 commits May 15, 2024 15:34

Filter out empty list when loading cached data

8fb4e82

Fix the build

b9d2611

0nko enabled auto-merge May 15, 2024 20:50

hichamboushaba removed the status: do not merge Dependent on another PR, ready for review but not ready for merge. label May 16, 2024

0nko merged commit e4cc452 into trunk May 16, 2024
14 of 15 checks passed

0nko deleted the issue/11461-orders-card branch May 16, 2024 08:29

0nko mentioned this pull request May 21, 2024

[Dynamic Dashboard] Make order items clickable #11554

Merged

		@@ -165,6 +166,8 @@ class DashboardFragment :

		is FeedbackNegativeAction -> mainNavigationRouter?.showFeedbackSurvey()

		is NavigateToOrders -> mainNavigationRouter?.showOrders()

[Dynamic Dashboard] Orders card #11486

[Dynamic Dashboard] Orders card #11486

Uh oh!

Conversation

0nko commented May 10, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

dangermattic commented May 10, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

wpmobilebot commented May 10, 2024

The PR caused the following dependency changes:

Uh oh!

wpmobilebot commented May 10, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

hichamboushaba commented May 10, 2024

Uh oh!

0nko commented May 10, 2024

Uh oh!

hichamboushaba left a comment

Choose a reason for hiding this comment

Uh oh!

hichamboushaba May 10, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

hichamboushaba commented May 13, 2024

Uh oh!

0nko commented May 14, 2024

Uh oh!

0nko commented May 14, 2024

Uh oh!

hichamboushaba left a comment

Choose a reason for hiding this comment

Uh oh!

codecov-commenter commented May 15, 2024

Codecov Report

Uh oh!

Uh oh!

Uh oh!

0nko commented May 10, 2024 •

edited

Loading

dangermattic commented May 10, 2024 •

edited

Loading

wpmobilebot commented May 10, 2024 •

edited

Loading

hichamboushaba May 10, 2024 •

edited

Loading