Skip to content

Remove application password availablity check#25439

Merged
crazytonyli merged 1 commit intorelease/26.8from
bugfix/remove-site-check
Mar 24, 2026
Merged

Remove application password availablity check#25439
crazytonyli merged 1 commit intorelease/26.8from
bugfix/remove-site-check

Conversation

@crazytonyli
Copy link
Contributor

Description

The removed guard ensures only self-hosted site with an application password goes through. But that check is not necessary because the ApplicationPasswordRequiredView below will ensure an application password is created for viewing the post list.

This change fixes an issue that is reproducible as follows:

  1. Sign in a self-hosted site using account username and password.
  2. After signing in, disable XML-RPC.
  3. Tap the Posts or Pages.

The app should present the cpt-based post list. But tapping the Posts and Pages row does nothing.

@crazytonyli crazytonyli added this to the 26.8 ❄️ milestone Mar 24, 2026
@crazytonyli crazytonyli requested a review from kean March 24, 2026 07:52
@dangermattic
Copy link
Collaborator

1 Warning
⚠️ This PR is assigned to the milestone 26.8 ❄️. The due date for this milestone has already passed.
Please assign it to a milestone with a later deadline or check whether the release for this milestone has already been finished.

Generated by 🚫 Danger

@sonarqubecloud
Copy link

@wpmobilebot
Copy link
Contributor

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
ConfigurationRelease-Alpha
Build Number31717
VersionPR #25439
Bundle IDorg.wordpress.alpha
Commitfcf7201
Installation URL5676mq8c9prkg
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@wpmobilebot
Copy link
Contributor

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
ConfigurationRelease-Alpha
Build Number31717
VersionPR #25439
Bundle IDcom.jetpack.alpha
Commitfcf7201
Installation URL1gp6sns988kg0
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@crazytonyli crazytonyli merged commit abc61d0 into release/26.8 Mar 24, 2026
29 checks passed
@crazytonyli crazytonyli deleted the bugfix/remove-site-check branch March 24, 2026 20:13
github-merge-queue bot pushed a commit that referenced this pull request Mar 24, 2026
* Remove the Post Settings feature flag (#25428)

* Use default content types for document picker (#25438)

* Add "BETA" to the Custom Post Types based post list screens (#25436)

* Add "BETA" to the Custom Post Types based post list screens

* Update wordpress-rs

* Remove application password availablity check (#25439)

* Update strings for localization

* Update app translations – `Localizable.strings`

* Update WordPress metadata translations

* Bump version number

---------

Co-authored-by: Tony Li <tony.li@automattic.com>
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.

4 participants