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.

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

[Android 14 PR] Set remaining foreground service types #21047

Merged
merged 6 commits into from
Jul 16, 2024

Conversation

nbradbury
Copy link
Contributor

@nbradbury nbradbury commented Jul 10, 2024

Fixes #21039

This PR targets the feature/android-14 branch and updates the following services to specify their foreground service type as part of our Android 14 update:

  • SiteCreationService
  • LoginWpcomService

The following services already have this change:

  • UploadService
  • MediaDeleteService

To Test:

  • Test site creation and login on an Android 34+ device and verify there are no issues
  • Do the same on a pre-Android 34 device

Notes:

  • Even though LoginWpcomService is part of our login library, we declare the foreground service type for that service in WPAndroid's manifest so no changes are needed in the service itself (which already targets SDK 34). Also see this related login PR and this related WooCommerce PR.

Regression Notes

  1. Potential unintended areas of impact

    • Failures in SiteCreationService and LoginWpcomService
  2. What I did to test those areas of impact (or what existing automated tests I relied on)

    • Manual testing
  3. What automated tests I added (or what prevented me from doing so)

    • I have not added tests under the assumption that existing tests would suffice

PR Submission Checklist:

  • I have completed the Regression Notes.
  • I have considered adding accessibility improvements for my changes.
  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

Testing Checklist (strike-out the not-applying and unnecessary ones):

  • WordPress.com sites and self-hosted Jetpack sites.
    - [ ] Portrait and landscape orientations.
    - [ ] Light and dark modes.
    - [ ] Fonts: Larger, smaller and bold text.
    - [ ] High contrast.
    - [ ] Talkback.
    - [ ] Languages with large words or with letters/accents not frequently used in English.
    - [ ] Right-to-left languages. (Even if translation isn’t complete, formatting should still respect the right-to-left layout)
    - [ ] Large and small screen sizes. (Tablet and smaller phones)
    - [ ] Multi-tasking: Split screen and Pop-up view. (Android 10 or higher)

@nbradbury nbradbury added this to the 25.4 milestone Jul 10, 2024
@nbradbury nbradbury changed the title [Android 14 PR] Foreground service type [Android 14 PR] Set remaining foreground service types Jul 10, 2024
@wpmobilebot
Copy link
Contributor

wpmobilebot commented Jul 10, 2024

WordPress📲 You can test the changes from this Pull Request in WordPress by scanning the QR code below to install the corresponding build.
App NameWordPress WordPress
FlavorJalapeno
Build TypeDebug
Versionpr21047-05c8ab9
Commit05c8ab9
Direct Downloadwordpress-prototype-build-pr21047-05c8ab9.apk
Note: Google Login is not supported on these builds.

@wpmobilebot
Copy link
Contributor

wpmobilebot commented Jul 10, 2024

Jetpack📲 You can test the changes from this Pull Request in Jetpack by scanning the QR code below to install the corresponding build.
App NameJetpack Jetpack
FlavorJalapeno
Build TypeDebug
Versionpr21047-05c8ab9
Commit05c8ab9
Direct Downloadjetpack-prototype-build-pr21047-05c8ab9.apk
Note: Google Login is not supported on these builds.

@irfano
Copy link
Member

irfano commented Jul 10, 2024

@nbradbury, I have added shortService to the LoginWpcomService for WCAndroid. You can review my reasoning in the description of these PRs:

Can you review the Login-Flow PR? If it makes sense to you, we can use the shortService type on WPAndroid too.

@nbradbury
Copy link
Contributor Author

Can you review the Login-Flow PR? If it makes sense to you, we can use the shortService type on WPAndroid too.

@irfano I'm reviewing that right now! I agree that shortService makes more sense and I've updated this PR accordingly.

@nbradbury nbradbury marked this pull request as ready for review July 11, 2024 18:05
@nbradbury nbradbury requested review from irfano and a team July 12, 2024 11:46
…ile/WordPress-Android into issue/21039-foreground-service-type

Conflicts:
	build.gradle
Copy link

sonarcloud bot commented Jul 15, 2024

@nbradbury nbradbury removed the request for review from irfano July 16, 2024 11:46
@nbradbury nbradbury merged commit f7b320f into feature/android-14 Jul 16, 2024
20 checks passed
@nbradbury nbradbury deleted the issue/21039-foreground-service-type branch July 16, 2024 11:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants