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

Req: exclude page and ip address #3

Closed
awrog opened this issue Aug 2, 2022 · 16 comments
Closed

Req: exclude page and ip address #3

awrog opened this issue Aug 2, 2022 · 16 comments

Comments

@awrog
Copy link

awrog commented Aug 2, 2022

The plugin works really well and is definitely an alternative to Google analytics.

It would add great value:

  • if I could exclude a page or pages (via frontmatter)
  • if I could exclude my own ip address
@francodacosta
Copy link
Owner

@awrog so yu want a front matter setting to not process a page ?

and a list of ips to exclude from analytics ?

@awrog
Copy link
Author

awrog commented Aug 2, 2022

@francodacosta That would be great.

An example for the frontmatter:

page-stats:
    process: false

A list of ip's to exclude in the plugin page would be awsome.

@francodacosta
Copy link
Owner

@awrog I agree it would be useful, I will implement that

another thing i'm considering is to block for a certain time IP's that constantly hit 404 pages. like '/wp-login.php'

@awrog
Copy link
Author

awrog commented Aug 2, 2022

Nice, mostly bots hoping it's WordPress ;-)
Excluding 404 / errorpage would be a nice addition (and also, there are other tools to check dead links)

@awrog
Copy link
Author

awrog commented Aug 2, 2022

Clickable pagetitles would also be nice (convert title attribute to href)

@francodacosta
Copy link
Owner

@awrog can you look at https://github.com/francodacosta/grav-plugin-page-stats/releases/tag/1.1.0 ?

should allow to exclude ips (either by ip or regular expression) and exclude pages via front matter

@awrog
Copy link
Author

awrog commented Aug 5, 2022

@francodacosta Simply brilliant! Also the feature to hide the unique visitors (I removed them myself ;-) )
I also changed the number of rows that are being displayed ($limit = 10 => 20)
Would be nice if I could specify that too in the plugin page. (no biggie, juist a 'nice to have')
All in all, great additions!

@francodacosta
Copy link
Owner

@awrog have a look at 1.2.0

I added the hability to:

  • enable/disable each widget
  • configure how many columns each widget uses in the grid
  • records to fetch for top pages, most viewed pages and top users

@awrog
Copy link
Author

awrog commented Aug 5, 2022

@francodacosta Impressive, really perfect.
The only thing I customize, is
{{s.page_title}}
I have a lot of pages with the same title, (and quite an extensive side menu (https://github.com/hibbitts-design/grav-theme-learn2-git-sync) )

Keep up the good work!!

@francodacosta
Copy link
Owner

francodacosta commented Aug 5, 2022

@awrog you want the url instead of the title ?

@francodacosta
Copy link
Owner

@awrog have a look at 1.3.1

this version needs a db migration as it collects browser and os info
it should happen automatically, if not make sure you have an emplty file on <plugin folder>/data/migrations/MUST_MIGRATE

@awrog
Copy link
Author

awrog commented Aug 5, 2022

@awrog you want the url instead of the title ?

A clickable title would be fine
I see Github interpreted by code, I meant:

<span title="{{s.route}}"><a href="{{s.route}}">{{s.page_title}}</a></span>

image

@francodacosta
Copy link
Owner

@awrog I might put a link (icon) to take you to the page.

My plan for when you click the page, is to go to a detailed stats page for that page

@awrog
Copy link
Author

awrog commented Aug 5, 2022

@awrog I might put a link (icon) to take you to the page.

My plan for when you click the page, is to go to a detailed stats page for that page

Nice, the url is awkward since it can be long (sometimes very long)
Title is just fine

image

=> In the plugin page, a title is missing

image

@awrog
Copy link
Author

awrog commented Aug 5, 2022

v1.5.0 all perfect now!

@awrog
Copy link
Author

awrog commented Aug 8, 2022

@francodacosta You've been busy I see ;-)
The page details are awsome!

The fact that you have to click on the pagename is not very intuitive for such a great feature.
Might I suggest a swap:

  • click on the pagename = open the page
  • click on the icon left from the pagename (e.g. fa-solid fa-chart-line or fa-solid fa-chart-column) opens the details page

Just my two cents...

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

No branches or pull requests

2 participants