Skip to content

Create Analytics user type (CRASM- 2144)#769

Merged
rapidray12 merged 75 commits intodevelopfrom
matomo
Jun 16, 2025
Merged

Create Analytics user type (CRASM- 2144)#769
rapidray12 merged 75 commits intodevelopfrom
matomo

Conversation

@Matthew-Grayson
Copy link
Contributor

@Matthew-Grayson Matthew-Grayson commented Jan 22, 2025

🗣 Description

  • Added an analytics user type to the backend models and schemas.
  • Added an analytics user type to the frontend user levels.
  • Configured role-based access control to grant analytics users access to Matomo-related routes.
  • Updated frontend components to recognize and properly authorize analytics users.

💭 Motivation and context

This PR introduces a new analytics role, enabling Customer Success team members to access analytics data returned from Matomo. The intent is to support the Customer Success team's need to evaluate user engagement and metrics for continuous improvement.

This resolves CRASM- 2144

📸 Screenshot

Added Analytics to User Form

Screenshot 2025-06-13 at 9 57 32 AM

Analytics View on User Registration

Screenshot 2025-06-13 at 9 58 34 AM

Matomo button for Analytics users

Screenshot 2025-06-13 at 10 09 05 AM

Matomo Dashboard

Screenshot 2025-06-13 at 11 06 12 AM Screenshot 2025-06-13 at 11 07 24 AM

🧪 Testing

✅ Pre-approval checklist

  • This PR has an informative and human-readable title.
  • Changes are limited to a single goal - eschew scope creep!
  • All future TODOs are captured in issues, which are referenced
    in code comments.
  • All relevant type-of-change labels have been added.
  • I have read the CONTRIBUTING document.
  • These code changes follow cisagov code standards.
  • All relevant repo and/or project documentation has been updated
    to reflect the changes in this PR.
  • Tests have been added and/or modified to cover the changes in this PR.
  • All new and existing tests pass.

✅ Pre-merge checklist

  • Revert dependencies to default branches.
  • Finalize version.

✅ Post-merge checklist

  • Create a release.

@chrtorres chrtorres changed the title Matomo Create Analytics user type (CRASM- 2144) Jun 13, 2025
@chrtorres chrtorres marked this pull request as ready for review June 13, 2025 04:05
rapidray12
rapidray12 previously approved these changes Jun 13, 2025
Copy link
Collaborator

@rapidray12 rapidray12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed in a group meeting

cduhn17
cduhn17 previously approved these changes Jun 13, 2025
Copy link
Collaborator

@cduhn17 cduhn17 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - review done in group. Has failing pytest

@chrtorres chrtorres dismissed stale reviews from cduhn17 and rapidray12 via e9712b4 June 16, 2025 14:00
@rapidray12 rapidray12 self-requested a review June 16, 2025 14:48
@rapidray12 rapidray12 merged commit 87d79f2 into develop Jun 16, 2025
20 of 21 checks passed
@rapidray12 rapidray12 deleted the matomo branch June 16, 2025 14:50
aloftus23 added a commit that referenced this pull request Jun 16, 2025
This reverts commit 87d79f2, reversing
changes made to 451202c.
@rapidray12 rapidray12 restored the matomo branch June 16, 2025 16:04
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.

6 participants