Skip to content
This repository was archived by the owner on Dec 19, 2023. It is now read-only.

Feature/aliased scalars #659

Conversation

BlasiusSecundus
Copy link

Define aliased scalars from config, similarly to extended scalars.

TODO before ready:

  • update documentation
  • improve test suite (add full integration tests)
  • manual testing
  • maybe some refactoring

At this stage it only works for standard and extended scalars (not
custom scalar beans). Also moved GraphQLExtendedScalarsInitializer
to separate subfolder, as it is a not Web Servlet implementation
specific feature.
@BlasiusSecundus BlasiusSecundus force-pushed the feature/aliased-scalars branch from 6e5b8bc to f685ce4 Compare January 30, 2022 15:45
@BlasiusSecundus BlasiusSecundus marked this pull request as ready for review February 2, 2022 18:57
Copy link
Member

@oliemansm oliemansm left a comment

Choose a reason for hiding this comment

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

Nice work @BlasiusSecundus!

@oliemansm oliemansm added this to the 12.0.1 milestone Mar 30, 2022
@oliemansm oliemansm merged commit 3945dbc into graphql-java-kickstart:master Apr 2, 2022
@BlasiusSecundus BlasiusSecundus deleted the feature/aliased-scalars branch May 11, 2022 19:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants