Skip to content

Conversation

pan93412
Copy link
Member

@pan93412 pan93412 commented Sep 29, 2025

CleanShot 2025-09-29 at 21 46 01@2x

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

linear bot commented Sep 29, 2025

Copy link
Contributor

vercel bot commented Sep 29, 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 29, 2025 1:49pm

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 adds a database relationship figure feature that displays an ER diagram for SQL challenge questions. The implementation includes a new GraphQL query to fetch database relationship figures and a React component to render them.

  • Adds DatabaseRelationship GraphQL query for fetching database relationship diagrams
  • Creates a new component to display the relationship figure with image and caption
  • Integrates the component into the challenge IDE layout

Reviewed Changes

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

File Description
gql/graphql.ts Adds generated TypeScript types for the new DatabaseRelationship query
gql/gql.ts Registers the new GraphQL query in the document mapping
app/(app)/challenges/[id]/_components/ide/index.tsx Integrates the DatabaseRelationship component and updates skeleton height
app/(app)/challenges/[id]/_components/ide/database-relationship.tsx New component that displays database relationship diagrams with captions

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

@pan93412 pan93412 merged commit c6c07b0 into main Sep 29, 2025
4 checks passed
@pan93412 pan93412 deleted the pan93412/dbp-69-「作答介面」缺少關聯圖 branch September 29, 2025 13:51
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