Skip to content

Refine SDKs/server-side libraries setting copy and add organisation block list example text, PG-5323 - #187

Merged
snake14 merged 1 commit into
5.x-devfrom
PG-5323-sdks-and-server-side-libraries-helptext
Aug 7, 2026
Merged

Refine SDKs/server-side libraries setting copy and add organisation block list example text, PG-5323#187
snake14 merged 1 commit into
5.x-devfrom
PG-5323-sdks-and-server-side-libraries-helptext

Conversation

@snake14

@snake14 snake14 commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Description

Follow-up to #186, refining settings copy based on UX review feedback:

  • The "Block tracking requests from server-side libraries" setting is renamed to "Block traffic from SDKs and server-side libraries". Its help text now clarifies that the option is only meant for sites tracked via the JavaScript Tracker or Matomo Tag Manager, and must not be enabled when tracking data is sent through the Matomo PHP/Java/Python/Android/iOS SDKs or other non-JavaScript libraries. Only the "Note:" label is bold, matching the organisation block list help text.

  • The organisation block list textarea now shows example text ("Cloud Hosting Provider A" / "Cloud Hosting Provider B") while the list is empty, matching the example IPs shown by the IP allow/block list fields.

Since the label and help text changed meaning, they are introduced as new translation keys instead of editing the existing keys in place, so existing translations of the old wording are not shown for the new text.

No version bump: this rides along in the unreleased 5.2.0.

Issue No

PG-5323

Steps to Replicate the Issue

  1. Go to Administration → General settings → TrackingSpamPrevention.
  2. Check the label and help text of the "Block traffic from SDKs and server-side libraries" setting.
  3. Enable "Block tracking requests from the cloud" and clear the organisation block list to see the example text.

Checklist

  • [✔] Tested locally or on demo2/demo3?
  • [✔] New test case added/updated?
  • [✔] 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
  • [NA] Documentation updated?

@snake14
snake14 marked this pull request as ready for review August 7, 2026 02:34
@snake14
snake14 requested a review from a team August 7, 2026 02:34

@AltamashShaikh AltamashShaikh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@snake14
snake14 merged commit bc663c5 into 5.x-dev Aug 7, 2026
8 of 9 checks passed
@snake14
snake14 deleted the PG-5323-sdks-and-server-side-libraries-helptext branch August 7, 2026 02:42
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