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

Analytics improvements #84

Merged
merged 2 commits into from
Sep 6, 2023
Merged

Analytics improvements #84

merged 2 commits into from
Sep 6, 2023

Conversation

fxprunayre
Copy link

to track link click by protocol and provide a sample matomo config which better works with single page app.

@josegar74 josegar74 merged commit 13f467f into GeoCat:analytics Sep 6, 2023
2 of 3 checks passed
josegar74 added a commit that referenced this pull request Jun 7, 2024
* Analytics services integration

* Analytics services integration - add settings help

* Analytics services integration - move configuration to config.properties

* Analytics services integration - better name for web analytics service

* Analytics improvements (#84)

* Analytics services integration / Send events by protocol

* Analytics services integration / Add example config for matomo.

* Update web/src/main/webResources/WEB-INF/config.properties

Co-authored-by: François Prunayre <[email protected]>

* Test / Fix bean initialization

Failing test were:

```
15:47:17,320 [INFO] Results:
15:47:17,320 [INFO] 
Error: 7,320 [ERROR] Failures: 
Error: 7,320 [ERROR]   AlternateLogoForPdfExportTest.whenGeneratingPdfWithPropertyNotSetSiteLogoIsUsed:114 Status expected:<200> but was:<400>
Error: 7,321 [ERROR]   AlternateLogoForPdfExportTest.whenGeneratingPdfWithPropertySetPdfLogoIsUsed:74 Status expected:<200> but was:<400>
Error: 7,321 [ERROR]   AlternateLogoForPdfExportTest.whenNotGeneratingPdfWithPropertySetSiteLogoIsUsed:93 Status expected:<200> but was:<400>
Error: 7,321 [ERROR] Errors: 
Error: 7,321 [ERROR]   FormatterApiIntegrationTest.testExec:97 » XPath Exception in extension functio.
```

---------

Co-authored-by: François Prunayre <[email protected]>
Co-authored-by: Juan Luis Rodríguez Ponce <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants