Skip to content

fix(compatibility): - Optimized enum value processing to match Prisma…#235

Closed
javan-b wants to merge 2 commits intounlight:masterfrom
javan-b:master
Closed

fix(compatibility): - Optimized enum value processing to match Prisma…#235
javan-b wants to merge 2 commits intounlight:masterfrom
javan-b:master

Conversation

@javan-b
Copy link
Copy Markdown
Contributor

@javan-b javan-b commented Feb 21, 2025

BREAKING CHANGE:
This change ensures that enum values are processed in a way that aligns with Prisma's engine, maintaining consistency in documentation handling and filtering out unnecessary entries.- Added specific handling for @deprecated documentation, mapping it to - Updated the registerEnum function to properly filter out empty entries in the valuesMap based on enum value documentation.- Ensured that enum values without description or deprecationReason are excluded from the valuesMap when registering with registerEnumType.the deprecationReason field in line with Prisma Engine behavior.- Improved compatibility with Prisma's enum value documentation format by ensuring only relevant metadata is included in the GraphQL schema.

…'s expected documentation structure.

BREAKING CHANGE:
This change ensures that enum values are processed in a way that aligns with Prisma's engine,
maintaining consistency in documentation handling and filtering out unnecessary entries.- Added
specific handling for `@deprecated` documentation, mapping it to - Updated the `registerEnum`
function to properly filter out empty entries in the `valuesMap` based on enum value documentation.-
Ensured that enum values without `description` or `deprecationReason` are excluded from the
`valuesMap` when registering with `registerEnumType`.the `deprecationReason` field in line with
Prisma Engine behavior.- Improved compatibility with Prisma's enum value documentation format by
ensuring only relevant metadata is included in the GraphQL schema.
@javan-b javan-b closed this by deleting the head repository May 10, 2025
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