Skip to content
This repository was archived by the owner on Sep 15, 2025. It is now read-only.

Conversation

@diegoreymendez
Copy link
Contributor

@diegoreymendez diegoreymendez commented Sep 14, 2021

Description

Adds the organization ID data field from remote endpoints to blogs.

Associated PRs:

WPiOS: wordpress-mobile/WordPress-iOS#17154

Testing Details

Use the testing steps from the associated WPiOS PR.

  • Please check here if your pull request includes additional test coverage.

Copy link
Contributor

@Gio2018 Gio2018 left a comment

Choose a reason for hiding this comment

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

:shipit:

@diegoreymendez diegoreymendez merged commit f4ca2f9 into develop Sep 16, 2021
@diegoreymendez diegoreymendez deleted the try/add-organization-id-to-blogs branch September 16, 2021 08:13
mokagio added a commit to wordpress-mobile/WordPressMocks that referenced this pull request Sep 17, 2021
Relates to the changes in:

- wordpress-mobile/WordPressKit-iOS#445
- wordpress-mobile/WordPress-iOS#17154

By looking at the changes in the PRs listed above, I'm guessing the key
has been in the response from the live API for a while, but we never
updated this particular mock response because we hadn't needed the value
till now.

I noticed that the Jetpack version of me/sites does include
`organization_id`, so we can say that the value had been in the response
at least since May 2021.
See
caa9d47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants