Skip to content

Conversation

@adalpari
Copy link
Contributor

@adalpari adalpari commented Jan 8, 2026

Description

Updates the Application Password experimental feature descriptions to clarify that new sites already use Application Password by default, while the toggle only affects existing sites.

Screenshot 2026-01-08 at 16 58 06 Screenshot 2026-01-08 at 16 55 13

Testing instructions

Verify experimental feature description:

  1. Open the app and go to Me → App Settings → Experimental Features
  • Verify the Application Password description reads: "Enable Application Password for existing self-hosted sites. New sites use it by default."

Verify info dialog description:

  1. Toggle the Application Password feature ON
  2. Read the info dialog that appears
  • Verify the second paragraph reads: "New sites added to the app are already using application passwords. Enabling this feature will also apply it to your existing sites."

@dangermattic
Copy link
Collaborator

1 Warning
⚠️ PR is not assigned to a milestone.

Generated by 🚫 Danger

@wpmobilebot
Copy link
Contributor

wpmobilebot commented Jan 8, 2026

App Icon📲 You can test the changes from this Pull Request in Jetpack by scanning the QR code below to install the corresponding build.
App NameJetpack
FlavorJalapeno
Build TypeDebug
Versionpr22468-09e9d90
Commit09e9d90
Direct Downloadjetpack-prototype-build-pr22468-09e9d90.apk
Note: Google Login is not supported on these builds.

@wpmobilebot
Copy link
Contributor

wpmobilebot commented Jan 8, 2026

App Icon📲 You can test the changes from this Pull Request in WordPress by scanning the QR code below to install the corresponding build.
App NameWordPress
FlavorJalapeno
Build TypeDebug
Versionpr22468-09e9d90
Commit09e9d90
Direct Downloadwordpress-prototype-build-pr22468-09e9d90.apk
Note: Google Login is not supported on these builds.

@adalpari adalpari requested a review from nbradbury January 8, 2026 16:38
@nbradbury nbradbury self-assigned this Jan 8, 2026
@nbradbury
Copy link
Contributor

For this dialog, do we need to clarify it's for new self-hosted sites?

533454053-3d787267-d772-42fa-9686-b41666d04287

@adalpari
Copy link
Contributor Author

adalpari commented Jan 8, 2026

For this dialog, do we need to clarify it's for new self-hosted sites?

Do you mean to change it by "... will also apply it to your existing self-hosted sites"?

@nbradbury
Copy link
Contributor

Do you mean to change it by "... will also apply it to your existing self-hosted sites"?

Nope, I meant where it says "New sites added to the app..." (in the section you outlined in red). Should that be "New self-hosted sites added to the app...". I don't have strong feelings either way so I'm happy to approve and leave to you to change it if you feel it's necessary.

Copy link
Contributor

@nbradbury nbradbury left a comment

Choose a reason for hiding this comment

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

Looks good! :shipit:

@adalpari adalpari enabled auto-merge (squash) January 9, 2026 08:06
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 9, 2026

@adalpari adalpari merged commit c0a01fb into trunk Jan 9, 2026
19 of 22 checks passed
@adalpari adalpari deleted the feat/clarifying-Applicarion-Password-experimental-feature-flag branch January 9, 2026 08:20
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.

5 participants