Skip to content

Conversation

@crazytonyli
Copy link
Contributor

Description

Fixes https://linear.app/a8c/issue/CMM-940.

Testing instructions

Verify CMM-940 is fixed. There are steps to reproduce the bug in the Linear issue.

@crazytonyli crazytonyli added this to the 26.5 milestone Nov 9, 2025
@crazytonyli crazytonyli requested review from jkmassel and kean November 9, 2025 21:13
@dangermattic
Copy link
Collaborator

dangermattic commented Nov 9, 2025

1 Warning
⚠️ This PR is assigned to the milestone 26.5. 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

@wpmobilebot
Copy link
Contributor

wpmobilebot commented Nov 9, 2025

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 Number29800
VersionPR #24976
Bundle IDcom.jetpack.alpha
Commitbec278c
Installation URL61gm4u5ec23mo
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@wpmobilebot
Copy link
Contributor

wpmobilebot commented Nov 9, 2025

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 Number29800
VersionPR #24976
Bundle IDorg.wordpress.alpha
Commitbec278c
Installation URL0cmd1qhm936r0
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

Copy link
Contributor

@jkmassel jkmassel left a comment

Choose a reason for hiding this comment

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

What happens if this is applied to a pre-existing site connected via XML-RPC? How does it work?

* [*] Add "Taxonomies" to Site Settings [#24955]
* [*] Update "Categories" picker to indicate multiple selection [#24952]
* [*] Fix a bug where the app can't access some Jetpack connected sites [#24976]
* [*] Add support for editing custom taxonomy terms from "Post Settings" [#24964]
Copy link
Contributor

Choose a reason for hiding this comment

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

This is from a different PR, I think

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, it is. I realized the missing release note when working on this PR. I thought I'd just add it here instead of creating a PR with one line change.

Co-authored-by: Jeremy Massel <[email protected]>
@crazytonyli
Copy link
Contributor Author

crazytonyli commented Nov 17, 2025

@jkmassel Good catch! The expected behaviour is that the existing self-hosted site should be merged into the newly signed-in WP.com sites. That's not the case before the changes in bac2464.

@crazytonyli crazytonyli force-pushed the bugfix/CMM-940-jetpack-sites-on-wpcom branch from 847f195 to bac2464 Compare November 17, 2025 03:30
@sonarqubecloud
Copy link

@crazytonyli crazytonyli requested a review from jkmassel November 19, 2025 22:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants