-
Notifications
You must be signed in to change notification settings - Fork 24
Allow config errors to be display to users #114
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
base: trunk
Are you sure you want to change the base?
Conversation
Shows errors when user is logged in, and has appropriate capabilities. Fixes #85.
Hi Paul! Can you share brief instructions in the PR description to test the PR? |
Most of the details are in the original issue: #85 (comment) but can adjust the PR if needed a bit later. |
I would suggest adding, in general, a short section with steps to test it and the expected outcome with and without the PR, similar to Gutenberg PRs. Even though the original ticket wasn't too hard to follow, it still required me to read all the comments to get the context 😅 |
Added in test instruction. |
For documentation purposes, note that I had to do some (obvious) extra steps:
With this branch, the error message appears indeed. However, I also see it when using the shortcode to display a Text Field, which, in theory, is supported by the shortcode. Am I missing something? |
🤔 presumably this works as expected without the patch? |
Shows errors when user is logged in, and has appropriate capabilities.
Fixes #85.
Testing instructions
Note: considered adding this to (
secure-custom-fields/includes/api/api-template.php
Line 1053 in a1257ed