Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Analytics API v4 #475

Merged
merged 68 commits into from
Apr 6, 2023
Merged

Analytics API v4 #475

merged 68 commits into from
Apr 6, 2023

Conversation

timvandijck
Copy link
Member

@timvandijck timvandijck commented Mar 17, 2023 •
edited
Loading

Important to-do's:

  • Rework to leverage the new API and Analytics Data package.
  • Reimplement methods with the new API.
  • Tests
  • Update installation docs
  • Update usage docs

Useful documentation:

@timvandijck timvandijck requested a review from freekmurze March 17, 2023 14:32
timvandijck and others added 26 commits March 17, 2023 16:02
@timvandijck
fetchVisitorsAndPageViews implementation
252e626
@timvandijck
Implement fetchTotalVisitorsAndPageViews
6cfaa4e
@freekmurze
use pint
9498fa9
@freekmurze @github-actions
Fix styling
fa06562
@freekmurze
use default namespace
a8a56ca
@freekmurze
Merge branch 'analytics-api-v4' of github.com:spatie/laravel-analytic…
8db0f68 
…s into analytics-api-v4
@freekmurze
formatting
8b87a62
@freekmurze
nitpick
c59ff8d
@freekmurze
improve docblock
f4ea915
@timvandijck
Merge
8f20d8d
@freekmurze
nitpicks
fe5c391
@freekmurze
nitpicks
8651883
@freekmurze
Merge branch 'analytics-api-v4' of github.com:spatie/laravel-analytic…
a662f30 
…s into analytics-api-v4
@freekmurze @github-actions
Fix styling
6ba9791
@freekmurze
update deps
3867ed3
@freekmurze
nitpicks
0c273ff
@freekmurze
Merge branch 'analytics-api-v4' of github.com:spatie/laravel-analytic…
54a4132 
…s into analytics-api-v4
@timvandijck
Add limit to fetchMostVisitedPages
5cffdb6
@freekmurze
restructure tests
b31d331
@freekmurze
restructure tests
653b1e0
@freekmurze
Merge branch 'analytics-api-v4' of github.com:spatie/laravel-analytic…
012f4b9 
…s into analytics-api-v4
@timvandijck
Implement fetchTopReferrers
52e0aad
@timvandijck @github-actions
Fix styling
02cedf0
@freekmurze
improve test names
1b26af3
@freekmurze
Merge branch 'analytics-api-v4' of github.com:spatie/laravel-analytic…
f347b2e 
…s into analytics-api-v4
@timvandijck
Update docs
43c6af8
@timvandijck timvandijck marked this pull request as ready for review March 24, 2023 12:24
@freekmurze freekmurze merged commit 5ed2684 into main Apr 6, 2023
@jcsoriano jcsoriano mentioned this pull request Apr 13, 2023
Closed
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@freekmurze freekmurze Awaiting requested review from freekmurze

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@timvandijck @freekmurze