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.

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

Add options page for tampering with Jetpack's constants #14

Merged
merged 4 commits into from
Jul 12, 2018

Conversation

oskosk
Copy link
Collaborator

@oskosk oskosk commented Jul 10, 2018

Proposal for addressing Automattic/jurassic.ninja#125

Changes introduced by this Pull Request

It will add a Settings -> Jetpack Constants to every launched site if Jetpack is at least active on the site.

Testing instructions

  1. Launch a regular JN site
  2. ssh to it, step into the public dir and install the version from this branch
     cd  ~/apps/$USER/public
     wp plugin install --force --activate https://github.com/Automattic/companion/archive/add/jetpack-constants.zip
  3. Visit wp-admin -> Settings and confirm that you can access the settings page this PR is introduceing under the Settings -> Jetpack Constants menu item.

image

@oskosk oskosk changed the title Add options page for tampering with Jetpack's consntants Add options page for tampering with Jetpack's constants Jul 10, 2018
@oskosk oskosk force-pushed the add/jetpack-constants branch from 0b50d23 to e3ec51f Compare July 10, 2018 21:19
@oskosk oskosk force-pushed the add/jetpack-constants branch from 56e1a9e to a04b4b3 Compare July 10, 2018 23:03
@oskosk oskosk force-pushed the add/jetpack-constants branch from a04b4b3 to 32ca769 Compare July 10, 2018 23:12
@oskosk oskosk force-pushed the add/jetpack-constants branch from 32ca769 to 9cbb63d Compare July 10, 2018 23:19
@simison
Copy link
Member

simison commented Jul 11, 2018

I tested this and it works brilliantly 👍

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.

2 participants