Skip to content

Conversation

@yehezkielgunawan
Copy link
Owner

Description

Linked Issues

Preview or Screenshot

@yehezkielgunawan yehezkielgunawan self-assigned this Jun 2, 2025
@vercel
Copy link

vercel bot commented Jun 2, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
vercel-og ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 2, 2025 1:40pm

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jun 2, 2025

Copy link

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 addresses issues with the edge runtime by removing unused font-loading code and updating styling in the OG image API endpoint, and by removing turbopack flags from npm scripts.

  • Removed asynchronous font preloading in src/pages/api/og.tsx and updated inline styling for a more modern look.
  • Updated package.json scripts to remove turbopack-related flags.

Reviewed Changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
src/pages/api/og.tsx Removed font data preloading and updated styling for improved consistency.
package.json Removed turbopack flags from the dev and build scripts.
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported
Comments suppressed due to low confidence (2)

src/pages/api/og.tsx:29

  • The h1 element is still referencing 'FiraSans-Bold' in its inline style, but the font data for FiraSans-Bold has been removed. Consider updating the font reference or adding the necessary font data to avoid unintended font fallback.
fontFamily: "FiraSans-Bold"

package.json:7

  • The removal of the '--turbopack' flag from the 'dev' and 'build' scripts may affect build performance. Please confirm that this change was intentional and that performance has been validated.
"dev": "next dev",

@yehezkielgunawan yehezkielgunawan merged commit 3a69b2c into main Jun 2, 2025
5 checks passed
@yehezkielgunawan yehezkielgunawan deleted the fix-edge-runtime branch June 2, 2025 13:42
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.

2 participants