Skip to content

Conversation

@AlixANNERAUD
Copy link

@AlixANNERAUD AlixANNERAUD commented Sep 3, 2025

Hi 👋,
Thanks for this awesome work on this theme.

This pull request adds support for specifying a custom script URL for Umami Analytics, allowing users to override the default region-based script. The documentation and analytics partial have been updated to reflect this new option.

This is related to : #829 (comment)

Changes

Umami Analytics configuration improvements:

  • Added umamiAnalytics.script parameter to the configuration documentation (exampleSite/content/docs/configuration/index.md) to allow specifying a custom script URL for Umami Analytics.
  • Updated the Umami Analytics documentation to explain how to use the new script parameter for custom or proxied script URLs, and clarified its precedence over the region parameter (exampleSite/content/docs/partials/index.md).

Template logic update:

  • Modified the analytics partial (layouts/_partials/analytics.html) to use the custom script URL if provided, falling back to the region-based default if not.

Testing

This has been tested locally with the following settings.

@netlify
Copy link

netlify bot commented Sep 3, 2025

Deploy Preview for hugo-congo ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit cbbaf9d
🔍 Latest deploy log https://app.netlify.com/projects/hugo-congo/deploys/68b86cd5fb4bf50008dc2c82
😎 Deploy Preview https://deploy-preview-1103--hugo-congo.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions bot added the documentation Improvements or additions to documentation label Sep 3, 2025
@AlixANNERAUD
Copy link
Author

Updates ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant