Skip to content

ACHOO-134: Graph of Views/Visits per day#13

Merged
jbickar merged 10 commits into2.xfrom
ACHOO-134
Sep 26, 2025
Merged

ACHOO-134: Graph of Views/Visits per day#13
jbickar merged 10 commits into2.xfrom
ACHOO-134

Conversation

@jbickar
Copy link
Contributor

@jbickar jbickar commented Sep 25, 2025

READY FOR REVIEW

Summary

  • Adds a line graph for Views and Visits at /applications/<uuid>

Review Tasks

  1. Check out this branch
  2. Navigate to applications/8449683b-500e-4728-b70a-5f69d9e8a61a
  3. Enter date range and fetch data
  4. View graph

@jbickar jbickar requested a review from Copilot September 25, 2025 22:06
@vercel
Copy link

vercel bot commented Sep 25, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
churro Ready Ready Preview Comment Sep 26, 2025 7:58pm

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a line graph visualization for Views and Visits per day on the application detail page at /applications/<uuid>. The implementation includes granularity support for fetching daily data from the Acquia API and displays interactive charts using recharts.

  • Adds granularity parameter to API service methods for fetching daily data
  • Implements line charts for daily Views and Visits using recharts library
  • Updates API routes to handle granularity parameter and pass it through to service methods

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File Description
lib/acquia-api.ts Adds granularity parameter support to API service methods and updates caching
app/applications/[uuid]/page.tsx Implements daily data processing and line chart visualization components
app/api/acquia/visits/route.ts Updates visits API route to handle granularity parameter
app/api/acquia/views/route.ts Updates views API route to handle granularity parameter

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@jbickar jbickar added the ai-generated Code in this PR was partially or wholly generated by AI label Sep 25, 2025
@jbickar jbickar self-assigned this Sep 25, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@jbickar jbickar merged commit 9cb015c into 2.x Sep 26, 2025
2 checks passed
@jbickar jbickar deleted the ACHOO-134 branch September 26, 2025 19:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-generated Code in this PR was partially or wholly generated by AI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant