Skip to content

Conversation

@itsmeichigo
Copy link
Contributor

Part of WOOMOB-1123

Description

This PR updates the default application password to fetch the site username to sign the password with when user is authenticated with WPCom token. Changes include:

  • Triggers the request to GET wp/v2/users/me with edit context to get the site username.
  • Moves User model, mapper, and remote from Networking to NetworkingCore module to be reused in DefaultApplicationPasswordUseCase.

Testing steps

The application password generation and deletion will be added in a subsequent PR so just unit tests passing is sufficient.

Testing information

Added unit tests for fetching username in application password use case.

Screenshots

No UI changes.


  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

@itsmeichigo itsmeichigo added this to the 23.2 milestone Aug 26, 2025
@itsmeichigo itsmeichigo added the type: task An internally driven task. label Aug 26, 2025
@wpmobilebot
Copy link
Collaborator

wpmobilebot commented Aug 26, 2025

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

App NameWooCommerce iOS Prototype
Build Numberpr16037-e5890fc
Version23.1
Bundle IDcom.automattic.alpha.woocommerce
Commite5890fc
Installation URL19n57dng9t788
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

Copy link
Contributor

@RafaelKayumov RafaelKayumov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code-wise the wporg username preloading looks good.

@itsmeichigo itsmeichigo merged commit 75cc81e into trunk Aug 27, 2025
14 checks passed
@itsmeichigo itsmeichigo deleted the task/woomob-1123-fetch-wporg-username branch August 27, 2025 02:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: task An internally driven task.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants