Closed StevenDufresne closed 4 years ago
make.wordpresstest.org
I'm guessing that was a typo in the description, but just wanted to check if that URL was used in the db too?
these updates were manual, if we have a better way to grab a snapshot of production data, I'm sure that's a better approach.
There isn't a secure way to do that right now (related), so it's definitely best to do it manually. Any PRs that include dumps of production data could unintentionally expose sensitive data.
@iandunn
make.wordpresstest.org
I'm guessing that was a typo in the description, but just wanted to check if that URL was used in the db too?
Not sure what you mean here, but to clarify, the theme wasn't configured properly in the existing snap shot. I turned the theme on and added some dummy data.
There isn't a secure way to do that right now (related), so it's definitely best to do it manually. Any PRs that include dumps of production data could unintentionally expose sensitive data.
Yeah, that concern is fair.
Oh, I was expecting the URL to be make.wordpressorg.test, to match the others.
Description
I apologize for the 'breadth' of changes that I want to merge (mostly db). This wasn't initially created to be a PR but rather for another purpose. I do however, think the community would benefit from the updates.
Updates:
Now, these updates were manual, if we have a better way to grab a snapshot of production data, I'm sure that's a better approach. Since I am not aware of one, I hand rolled them :). These updates do not go deeper than the top level. Meaning, it doesn't guarantee that all the functionality in each theme works 100%.
Notes
The DB changes were pretty significant, they will be hard to read.