Skip to content

Conversation

pan93412
Copy link
Member

No description provided.

@pan93412 pan93412 self-assigned this Sep 24, 2025
Copy link

linear bot commented Sep 24, 2025

@pan93412 pan93412 force-pushed the pan93412/dbp-63-enable-apq-on-frontend branch from 2914da3 to 4c74d14 Compare September 24, 2025 16:21
Copy link
Contributor

vercel bot commented Sep 24, 2025

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

Project Deployment Preview Comments Updated (UTC)
dbplay-app Ready Ready Preview Comment Sep 24, 2025 4:34pm

@pan93412 pan93412 requested a review from Copilot September 24, 2025 16:29
Copy link
Contributor

@Copilot 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 enables Automatic Persisted Queries (APQ) for Apollo Client to optimize GraphQL query performance by caching query strings on the server and sending only hashes from the client. Additionally, it includes formatting improvements for percentage display in statistics components.

  • Implements APQ using PersistedQueryLink with SHA-256 hashing
  • Updates percentage calculations to display whole numbers instead of decimal precision
  • Adds crypto-hash dependency for SHA-256 hashing functionality

Reviewed Changes

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

File Description
package.json Adds crypto-hash dependency for SHA-256 hashing
lib/apollo.ts Implements APQ by adding PersistedQueryLink to Apollo Client configuration
app/(app)/statistics/_components/statistics/resolved-questions.tsx Updates percentage calculation to multiply by 100 for proper percentage display
app/(app)/statistics/_components/board/completed-questions.tsx Changes percentage formatting from 2 decimal places to whole numbers
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

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

Copy link
Contributor

@Copilot 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 5 out of 6 changed files in this pull request and generated no new comments.

Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

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

@pan93412 pan93412 merged commit 08c8a21 into main Sep 24, 2025
4 checks passed
@pan93412 pan93412 deleted the pan93412/dbp-63-enable-apq-on-frontend branch September 24, 2025 16:37
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