Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Next step of Photon transition #14009

Merged
merged 1 commit into from
Nov 12, 2019
Merged

Next step of Photon transition #14009

merged 1 commit into from
Nov 12, 2019

Conversation

kraftbj
Copy link
Contributor

@kraftbj kraftbj commented Nov 11, 2019

Follow-up to #12159

Changes proposed in this Pull Request:

  • Expand to 2/3rd of images the simpler Photon subdomain determination.

Is this a new feature or does it add/remove features to an existing part of Jetpack?

  • pMz3w-arH-p2

Testing instructions:

  • None really, but ensure Photon is still loading images.

Proposed changelog entry for your changes:

  • Photon: Expand number of images using the new subdomain determination function.

@kraftbj kraftbj added [Feature] Photon aka "Image CDN". Feature developed in the Image CDN package and shipped in multiple plugins [Status] Needs Review To request a review from fellow Jetpack developers. Label will be renamed soon. [Type] Janitorial labels Nov 11, 2019
@kraftbj kraftbj added this to the 8.0 milestone Nov 11, 2019
@kraftbj kraftbj requested a review from a team November 11, 2019 03:28
@kraftbj kraftbj self-assigned this Nov 11, 2019
@matticbot
Copy link
Contributor

Caution: This PR has changes that must be merged to WordPress.com
Hello kraftbj! These changes need to be synced to WordPress.com - If you 're an a11n, please commandeer, review, and approve D35278-code before merging this PR. Thank you!

@jetpackbot
Copy link
Collaborator

Thank you for the great PR description!

When this PR is ready for review, please apply the [Status] Needs Review label. If you are an a11n, please have someone from your team review the code if possible. The Jetpack team will also review this PR and merge it to be included in the next Jetpack release.

Scheduled Jetpack release: December 3, 2019.
Scheduled code freeze: November 26, 2019

Generated by 🚫 dangerJS against f15cab4

@jeherve jeherve added [Status] Ready to Merge Go ahead, you can push that green button! and removed [Status] Needs Review To request a review from fellow Jetpack developers. Label will be renamed soon. labels Nov 12, 2019
@kraftbj kraftbj merged commit 3ad1748 into master Nov 12, 2019
@kraftbj kraftbj deleted the photon/8-0-update branch November 12, 2019 17:36
@matticbot matticbot added [Status] Needs Changelog and removed [Status] Ready to Merge Go ahead, you can push that green button! labels Nov 12, 2019
jeherve added a commit that referenced this pull request Nov 15, 2019
@jeherve
Changelog: add #14009
e037999
jeherve added a commit that referenced this pull request Nov 25, 2019
@jeherve
8.0 Release: running changelog (#13964)
807ab89 
* 8.0 Release: running changelog

* Changelog: add #13921

* Changelog: add #13980

* Changelog: add #13905

* Changelog: add #13971

* Changelog: add #13984

* Changelog: add #14009

* Changelog: add #13620

* Remove things that will ship in 7.9.1

* Changelog: add 7.9.1 release (#14044)

* Changelog: add base for 7.9.1 release

* Update release date and post link

* Changelog: add #14066

* Update changelog for 7.9.1

* Changelog: add #13405

* Changelog: add #13841

* Changelog: add #13924

* Changelog: add #13986

* Changelog: add #14010, #14028, #14053, #14055.

* Changelog: add #14054

* Changelog: add #14031

* Changelog: add #14039

* Changelog: add #14050

* Changelog: add #14070

* Changelog: add #14082

* Changelog: add #14084

* Changelog: add #14111

* Changelog: add #13961

* Changelog: add #14047

* Changelog: add #14091

* Changelog: add #14108

* Changelog: add #14121
@kraftbj kraftbj mentioned this pull request Dec 6, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jeherve jeherve jeherve approved these changes

Assignees

@kraftbj kraftbj

Labels
[Feature] Photon aka "Image CDN". Feature developed in the Image CDN package and shipped in multiple plugins Touches WP.com Files [Type] Janitorial
Projects
None yet
Milestone
8.0
Development

Successfully merging this pull request may close these issues.
4 participants
@kraftbj @matticbot @jetpackbot @jeherve