Conversation
# Conflicts: # WordPress/src/main/java/org/wordpress/android/viewmodel/activitylog/ActivityLogViewModel.kt
|
You can trigger optional UI/connected tests for these changes by visiting CircleCI here. |
|
You can test the changes on this Pull Request by downloading the APK here. |
There was a problem hiding this comment.
Thanks for the clear instructions @malinajirka! I tested the scenarios (after changing fluxc version to the latest commit hash from the linked fluxc PR) and mostly everything worked as described:
Test filters 2 ✅
Test clear cache action ✅
Test cancelation of an ongoing request ✅
Disable ActivityLogFiltersFeatureConfig and make sure the activity log still works ✅
Good job!
Few observations when playing with the filters in "Test filters 1":
-
Selecting same day in the start and end date doesn't seem to filter the logs correctly:
Dec 21-22 -> log appears
Dec 21 -> empty screen
Dec 22 -> empty screenLog that appeared in Dec 21-22 should appear either in Dec 21 or Dec 22 but empty screen is shown for both the dates.
Video -
"Activity type name shown on the chip is not the same as the one selected, is that expected?
E.g. Choosing "Backup and Restore" displays "rewind"
Video -
Different no. of logs appear on the web and mobile for the same date range (timezone effect?):
Web: https://cldup.com/sxBP-XAmkH.png
Mobile: https://cldup.com/oWowe3YDDd.png
Used below sites for testing:
1, 2: pressable-jetpack-daily-backup
3: pressable-jetpack-complete
Parent issue #13268
Merge instructions
Known issues which will be fixed in upcoming PRs
To test:
Test filters 1Test filters 2Test Clear cache actionTest cancelation of an ongoing request*affect even the activity log without filters and these changes will go live with the next version of the app.PR submission checklist:
RELEASE-NOTES.txtif necessary.