Skip to content

Correct FAQ: config-overridden IP lists are hidden from the UI, not read-only - #183

Merged
snake14 merged 1 commit into
5.x-devfrom
fix-faq-config-override-wording
Jul 31, 2026
Merged

Correct FAQ: config-overridden IP lists are hidden from the UI, not read-only#183
snake14 merged 1 commit into
5.x-devfrom
fix-faq-config-override-wording

Conversation

@snake14

@snake14 snake14 commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Description

Corrects the FAQ wording for the ip_allow_list / ip_block_list config overrides: while an override is in place the setting is hidden from General Settings, it does not become read-only. The old wording caused confusion when the fields disappeared after adding the config values.

Issue No

Related to PG-3570 / PG-5272

Steps to Replicate the Issue

  1. Add ip_allow_list[] entries under [TrackingSpamPrevention] in config/config.ini.php and open General Settings.
  2. Expected (per current FAQ): the field becomes read-only.
  3. Actual: the field is hidden while the override is in place (standard behaviour for plugin settings).

Checklist

  • [NA] Tested locally or on demo2/demo3? (docs-only change)
  • [NA] New test case added/updated?
  • [NA] Are all newly added texts included via translation?
  • [NA] Are text sanitized properly? (Eg use of v-text v/s v-html for vue)
  • [✖] Version bumped?
  • [✔] I have understood, reviewed, and tested all AI outputs before use
  • [✔] All AI instructions respect security, IP, and privacy rules
  • [✔] Documentation updated?

🤖 Generated with Claude Code

…I, not read-only

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@snake14
snake14 requested a review from a team July 31, 2026 05:35
@snake14
snake14 merged commit bfe655d into 5.x-dev Jul 31, 2026
8 of 9 checks passed
@snake14
snake14 deleted the fix-faq-config-override-wording branch July 31, 2026 05:38
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