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

Update README.md. Proposed fix for issue #528 #718

Closed
wants to merge 1 commit into from

Conversation

reshmee011
Copy link
Contributor

Proposed fix for issue #528

Proposed fix for issue pnp#528
@pkbullock
Copy link
Collaborator

Hi @reshmee011 - just looking at this, did you ever get any values from the Usage object, i was testing out of curiosity and its not returning values for me, did you get the same?

@pkbullock
Copy link
Collaborator

pkbullock commented Jul 29, 2024

@reshmee011
Copy link
Contributor Author

Thanks @pkbullock for looking into it. I looked into Get-PnPSiteAnalyticsData which calls the endpoint getActivitiesByInterval under the hood, unfortunately it does not return the same type of data: hits and visits.

image

I don't remember ever seeing the usage.hits, usage.bandwidth and usage.visits returning any values. I did the CLI for M365 version and tried to keep the same logic and I did not do much testing or investigating into missing data.

@reshmee011 reshmee011 closed this Aug 7, 2024
@reshmee011 reshmee011 deleted the patch-1 branch August 7, 2024 08:20
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