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

[Enhancement] Counters for Number of Views #259

Closed
tobogdan opened this issue May 23, 2019 · 4 comments · Fixed by #1870
Closed

[Enhancement] Counters for Number of Views #259

tobogdan opened this issue May 23, 2019 · 4 comments · Fixed by #1870
Labels
enhancement New feature or request

Comments

@tobogdan
Copy link

As the title says, if it's possible to add a viewcount tracker for each item.

On what I am hosting (on 4.0.0), we'd like to know how many times certain items were viewed, so having some tracking mechanism for it would be great.

@ildyria ildyria added the enhancement New feature or request label May 24, 2019
@timo-reymann
Copy link
Contributor

timo-reymann commented Jun 4, 2023

Would it be a possibility to allow users injecting javascript? This way it would be pretty simple to use tools like Matomo or Google Analytics to get exactly that.

If thats an option would also take a look implementing it and creating a PR.

Sounds similar to #348 (comment)

@ildyria
Copy link
Member

ildyria commented Jun 6, 2023

@timo-reymann the ability to inject JS is already implemented:
#1697

@timo-reymann
Copy link
Contributor

@ildyria Ah alrighty, didnt see that. Unfortunately the CSP is not allowing matomo code execution never the less :/

@ildyria
Copy link
Member

ildyria commented Jun 6, 2023

You can fix that by changing this file: https://github.com/LycheeOrg/Lychee/blob/master/config/secure-headers.php
I think we would accept a Pull request providing an optional support for that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants