Skip to content

Conversation

@pan93412
Copy link
Member

  • chore: upgrade dependencies
  • feat: add create mutation to Point model
  • feat: add CreatePoint mutation

@pan93412 pan93412 self-assigned this Oct 23, 2025
@linear
Copy link

linear bot commented Oct 23, 2025

@pan93412 pan93412 requested a review from Copilot October 23, 2025 15:14
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 new GraphQL mutation for creating point records in the backend system. The main changes include:

  • Adding a createPoint mutation to the GraphQL schema with proper scope protection
  • Implementing the resolver to handle point record creation
  • Updating schema annotations to enable mutation generation
  • Upgrading several Go dependencies (posthog-go, google.golang.org/api)

Reviewed Changes

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

Show a summary per file
File Description
graph/user.resolvers.go Implements the CreatePoint mutation resolver
graph/user.graphqls Adds the createPoint mutation definition with point:write scope
graph/ent.graphqls Defines CreatePointInput for the mutation
ent/schema/point.go Enables mutation creation for Point model
ent/gql_mutation_input.go Generated code for CreatePointInput structure and methods
ent/internal/schema.go Auto-generated schema update reflecting Point model changes
go.mod Updates dependencies to newer versions
docs/scope.md Documents the new point:write scope

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

@pan93412 pan93412 merged commit b883806 into main Oct 23, 2025
2 checks passed
@pan93412 pan93412 deleted the pan93412/dbp-126-backend-新增-addpointrecord-mutation branch October 23, 2025 15:18
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