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

Feature: add analytics controller with endpoints for users, ontologies and pages #63

Merged
merged 1 commit into from
Dec 28, 2023

Conversation

syphax-bouazzouni
Copy link

@syphax-bouazzouni syphax-bouazzouni commented Dec 28, 2023

Prerequisites

Changes

  • Add /data/analytics/ontologies endpoint, to get ontologies visits evolution by time.
  • Add /data/analytics/users endpoint, to get users visits evolution by time.
  • Add /data/analytics/page_visits endpoint, to get top pages visited in the current month.

Copy link

codecov bot commented Dec 28, 2023

Codecov Report

Attention: 22 lines in your changes are missing coverage. Please review.

Comparison is base (72813a0) 71.17% compared to head (236adf5) 70.76%.

Files Patch % Lines
controllers/analytics_controller.rb 21.42% 22 Missing ⚠️
Additional details and impacted files
@@               Coverage Diff               @@
##           development      #63      +/-   ##
===============================================
- Coverage        71.17%   70.76%   -0.41%     
===============================================
  Files               64       65       +1     
  Lines             3393     3421      +28     
===============================================
+ Hits              2415     2421       +6     
- Misses             978     1000      +22     
Flag Coverage Δ
unittests 70.76% <26.66%> (-0.41%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@syphax-bouazzouni syphax-bouazzouni merged commit 422cbb5 into development Dec 28, 2023
2 of 4 checks passed
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.

1 participant