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

Improve fetching of settings #299

Merged
merged 2 commits into from Sep 29, 2016

Conversation

Projects
None yet
2 participants
@westonruter
Copy link
Contributor

commented Sep 29, 2016

No description provided.

@westonruter westonruter added this to the 0.8.2 milestone Sep 29, 2016

@westonruter westonruter merged commit 1d95e90 into develop Sep 29, 2016

3 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.01%) to 94.758%
Details

@westonruter westonruter deleted the bugfix/previewed-fetched-settings branch Sep 29, 2016

@valendesigns

This comment has been minimized.

Copy link
Member

commented on php/class-wp-customize-posts.php in 0df94bb Oct 18, 2016

@westonruter This has broken BOD. Specifically the values in postmeta settings are not being added to the control even though they are stored in the DB. So you see an empty input instead of the actual value.

This comment has been minimized.

Copy link
Contributor Author

replied Oct 18, 2016

Strange 😦 I'm not sure why that would be.

This comment has been minimized.

Copy link
Contributor Author

replied Oct 18, 2016

TBH, I can't remember exactly why I added it.

This comment has been minimized.

Copy link
Contributor Author

replied Oct 18, 2016

Issue should be fixed now, props @PatelUtkarsh: #309

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.