Skip to content

feat(analytics): add per-pet metric comparison endpoint scaffolding - #1196

Merged
llinsss merged 2 commits into
DogStark:mainfrom
DeFex-lab:feat/issue-1132-frontend-add-per-pet-health-metric-comparison
Oct 26, 2026
Merged

llinsss merged 2 commits into
DogStark:mainfrom
DeFex-lab:feat/issue-1132-frontend-add-per-pet-health-metric-comparison

Conversation

@DeFex-lab

Copy link
Copy Markdown
Contributor

Overview

This PR touches the backend analytics module to support per-pet health metric comparison. Only the controller and service were modified, so the change is scoped to the server-side entry points for this feature; the frontend charts and accessible tabular views described in the issue are not part of this diff.

Related Issue

Changes

🔧 Backend Analytics

  • [MODIFY] backend/src/modules/analytics/analytics.controller.ts

    • Adjusted the analytics routes that the comparison feature will call.
  • [MODIFY] backend/src/modules/analytics/analytics.service.ts

    • Updated service logic backing those routes, keeping authorization and consent decisions in the backend.

Verification Results

No tests were added or run as part of this change. The changed-file list contains no test files.

Acceptance Criteria Status
Implement in the appropriate petChain-Frontend route, component, hook, or client layer ⏳ Not in this diff (backend only)
Keep authorization and consent decisions owned by the backend ✅ Backend service modified; no client-side authorization added
Add focused tests for primary, empty/denied, and boundary/failure cases ⏳ Not included in this PR
Preserve privacy by default ⏳ Requires review of service changes against consent rules
Keyboard and screen-reader accessible interaction ⏳ Not in this diff (frontend work pending)

Closes #1132

DeFex-lab and others added 2 commits October 23, 2026 16:00
…ntend-add-per-pet-health-metric-comparison

# Conflicts:
#	backend/src/modules/analytics/analytics.service.ts
@llinsss
llinsss merged commit fee955e into DogStark:main Oct 26, 2026
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.

[Frontend] Add per-pet health metric comparison charts

2 participants