[GitHub] [maven-fluido-skin] kwin commented on pull request #22: [MSKINS-177] Support native apacheAnalyticsSiteId as macro

2022-04-20 Thread GitBox
kwin commented on PR #22: URL: https://github.com/apache/maven-fluido-skin/pull/22#issuecomment-1103524637 This deserves some documentation -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [maven-fluido-skin] kwin commented on pull request #22: [MSKINS-177] Support native apacheAnalyticsSiteId as macro

2022-04-19 Thread GitBox
kwin commented on PR #22: URL: https://github.com/apache/maven-fluido-skin/pull/22#issuecomment-1102767815 This should only be Matomo-specific, not ASF specific. Just make the full host a mandatory config option (next to site id) to make this feature usable outside ASF projects. -- This