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

'jetpack_get_available_modules' Filter Shows 'undefined' Setting for Photon #8810

Closed
mrjarbenne opened this issue Feb 10, 2018 · 3 comments
Closed

'jetpack_get_available_modules' Filter Shows 'undefined' Setting for Photon #8810

mrjarbenne opened this issue Feb 10, 2018 · 3 comments
Labels
Admin Page React-powered dashboard under the Jetpack menu [Feature] Photon aka "Image CDN". Feature developed in the Image CDN package and shipped in multiple plugins [Status] Needs Author Reply We would need you to make some changes or provide some more details about your PR. Thank you! [Type] Bug When a feature is broken and / or not performing as intended

Comments

@mrjarbenne
Copy link

I'm creating a separate ticket for this issue (as I should have done initially rather than tagging onto the bottom of this: #8778). The issue was also affecting Lazy Load images prior to this commit: #8780 but is still affecting Photon as seen in the image below:

2018-02-07_23-55-44

Jetpack Version 5.8
@jeherve jeherve added [Type] Bug When a feature is broken and / or not performing as intended [Feature] Photon aka "Image CDN". Feature developed in the Image CDN package and shipped in multiple plugins [Status] Needs Author Reply We would need you to make some changes or provide some more details about your PR. Thank you! Admin Page React-powered dashboard under the Jetpack menu labels Feb 12, 2018
@jeherve
Copy link
Member

jeherve commented Feb 12, 2018

To clarify, you are still experiencing an issue when using the current version of master, that includes the changes from #8780?

@jeherve
Copy link
Member

jeherve commented Feb 12, 2018

Related: #8800

@mrjarbenne
Copy link
Author

Just checked again with the latest master, and with the changes from #8780 the Photon settings are properly hidden. Sorry about the additional issue. Cheers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Admin Page React-powered dashboard under the Jetpack menu [Feature] Photon aka "Image CDN". Feature developed in the Image CDN package and shipped in multiple plugins [Status] Needs Author Reply We would need you to make some changes or provide some more details about your PR. Thank you! [Type] Bug When a feature is broken and / or not performing as intended
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jeherve @mrjarbenne