Skip to content

chore: remove orphaned Event GraphQL fields (slug, matchingAlgorithm, emailTemplate) - #65

Open
detail-app[bot] wants to merge 1 commit into
mainfrom
detail/dead-code/chore-remove-orphaned-event-graphql-fields-slug-ma-b8b39a
Open

detail-app[bot] wants to merge 1 commit into
mainfrom
detail/dead-code/chore-remove-orphaned-event-graphql-fields-slug-ma-b8b39a

Conversation

@detail-app

@detail-app detail-app Bot commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

Removed three @Field-decorated properties from the Event ObjectType that had
no backing Prisma column, no @FieldResolver, no production callers, and no
references in EventEditInput or resolvers. Because they were declared as
non-nullable @field(() => String), any GraphQL query selecting them would
have errored at runtime — they were already broken.

  • slug: never had an Event column in prisma/schema.prisma; the Survey column
    it relates to was added then dropped separately.
  • matchingAlgorithm / emailTemplate: no Prisma column, no resolver, dead
    since the initial 2021 scaffold.

History

  • Last materially changed January 2024 (05f7042)

Dead Code PRs can be configured here.

@augmentcode

augmentcode Bot commented Sep 22, 2026

Copy link
Copy Markdown
Contributor
🤖 Augment PR Summary

Summary: Removes the orphaned slug, matchingAlgorithm, and emailTemplate fields from the GraphQL Event object type.
Technical note: These non-null fields had no Prisma columns or field resolvers, so selecting them could fail at runtime; the change aligns the GraphQL schema with the persisted Event model.

🤖 Was this summary useful? React with 👍 or 👎

@augmentcode augmentcode Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Review completed. No suggestions at this time.

Comment augment review to trigger a new review at any time.

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