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

User Documentation >> Usage Stats #1101

Merged
merged 2 commits into from
May 2, 2019
Merged

User Documentation >> Usage Stats #1101

merged 2 commits into from
May 2, 2019

Conversation

Natkeeran
Copy link
Contributor

Adding basic documentation about web analytics/usage stats.

GitHub Issue: (link)

What does this Pull Request do?

  • Basic documentation about web analytics/usage stats integration in Islandora 8.

Additional Notes:

Consider adding this to user documentation.

Interested parties

@manez
@Islandora-CLAW/committers

Adding basic documentation about web analytics/usage stats.
@@ -0,0 +1,6 @@
Drupal 8 supports various web analytics integrations such as [google analytics](https://www.drupal.org/project/google_analytics) and [matomo](https://www.drupal.org/project/matomo). For privacy reasons, Islandora 8 integrates Matomo web analytics platforms by default.

The matomo server is installed here: http://localhost:8000/matomo . The default configurations can be found [here](http://localhost:8000/admin/config/system/matomo). You can login to the matomo server using username `admin` and password `islandora`. To see page views, logiin to matomo and go to Behaviour >> Pages.
Copy link
Member

Choose a reason for hiding this comment

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

Small typo: logiin -> login.

There's an extra space before the period after the link http://localhost:8000/matomo.

Capitalize 'matomo'? I think it qualifies as a proper noun in this context.

Copy link
Member

Choose a reason for hiding this comment

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

Also: I'd suggest writing out the link to default configs, so it's clear that it's going to localhost.

update as per @manez feedback
@manez
Copy link
Member

manez commented May 2, 2019

👍 here. @Islandora-CLAW/committers ?

@rosiel rosiel merged commit 10dbcbf into master May 2, 2019
@Natkeeran Natkeeran deleted the Natkeeran-patch-4 branch May 2, 2019 19:21
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.

3 participants