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

feat: add visits and shiny usage to metrics api #387

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

tdstein
Copy link
Collaborator

@tdstein tdstein commented Feb 19, 2025

A start of a change to expose each of the two current metrics endpoints independently. It needs tests and documentation.

However this isn't intended to and likely will not land: What's the use case for comparing them separately? It's a pain to manually coalesce them when analyzing user traffic, which is why I decided on the approach on main. Here's a summary of the differences I observed. https://docs.posit.co/connect/cookbook/metrics/viewing-content-usage-information/#discussion

Copy link

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
1824 1713 94% 0% 🟢

New Files

No new covered files...

Modified Files

File Coverage Status
src/posit/connect/metrics/metrics.py 86% 🟢
TOTAL 86% 🟢

updated for commit: 0169905 by action🐍

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant