Closed thomashorta closed 2 weeks ago
1 Warning | |
---|---|
:warning: | This PR is larger than 300 lines of changes. Please consider splitting it into smaller PRs for easier and faster reviews. |
Generated by :no_entry_sign: Danger
App Name | WordPress | |
Flavor | Jalapeno | |
Build Type | Debug | |
Version | pr20684-59ba75c | |
Commit | 59ba75c6f3ace25198d20d2c7c56d33c8edb2e37 | |
Direct Download | wordpress-prototype-build-pr20684-59ba75c.apk |
App Name | Jetpack | |
Flavor | Jalapeno | |
Build Type | Debug | |
Version | pr20684-59ba75c | |
Commit | 59ba75c6f3ace25198d20d2c7c56d33c8edb2e37 | |
Direct Download | jetpack-prototype-build-pr20684-59ba75c.apk |
Attention: Patch coverage is 40.64171%
with 111 lines
in your changes are missing coverage. Please review.
Project coverage is 40.39%. Comparing base (
5e50e18
) to head (59ba75c
). Report is 486 commits behind head on feature/tags-ia.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Issues
6 New issues
0 Accepted issues
Measures
0 Security Hotspots
No data about Coverage
2.9% Duplication on New Code
Fixes #20588
Move shared fetching logic to Repository and create ViewModel to use Repository to fetch posts for tags directly.
To Test:
reader_tags_feed
feature flag in Debug SettingsTags
feedRegression Notes
Potential unintended areas of impact
What I did to test those areas of impact (or what existing automated tests I relied on)
What automated tests I added (or what prevented me from doing so)
PR Submission Checklist:
RELEASE-NOTES.txt
if necessary.Testing Checklist (strike-out the not-applying and unnecessary ones):
N/A, the UI in this PR is completely throwaway and only there to easily test these changes.