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

INI format warning? #160

Closed
dannysummerlinjstartorg opened this issue Nov 6, 2020 · 1 comment
Closed

INI format warning? #160

dannysummerlinjstartorg opened this issue Nov 6, 2020 · 1 comment
Assignees
Milestone

Comments

@dannysummerlinjstartorg
Copy link

<textarea id="configpress-editor" class="configpress-editor" rows="10"><?php echo AAM_Core_ConfigPress::getInstance()->read(); ?></textarea>

Just a thought/recommendation - we had messed up our permissions by not having some ConfigPress options set, but then we we set them, it still didn't work. We realized we didn't have [aam] at the top to define the INI section - we've got it now, but it would have been great if there was a warning or even automatic addition of that tag to help make things clear. I can take a look at adding a JS catch but I just wanted to through the idea out there!

@aamplugin aamplugin self-assigned this Feb 15, 2021
@aamplugin aamplugin added this to the 6.7.4 - AAM milestone Feb 15, 2021
@aamplugin
Copy link
Owner

@dannysummerlinjstartorg very good point. Adding the default [aam] to the ConfigPress if it is empty.

vasyltech added a commit that referenced this issue Feb 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants