Extended scalar for Url implementation

[timpamungkas]

Trying to add Url extended scalars.
On build.gradle.

	implementation(platform('com.netflix.graphql.dgs:graphql-dgs-platform-dependencies:3.12.0'))
	implementation 'com.netflix.graphql.dgs:graphql-dgs-spring-boot-starter'
	implementation 'com.netflix.graphql.dgs:graphql-dgs-extended-scalars'

The schema

scalar Date
scalar Url

type MobileApp {
    name: String!
    version: String
    platform: [String]
    releaseDate: Date
    homepage: Url
}

Generated class by gradle plugin.

import java.lang.Object;
import java.lang.Override;
import java.lang.String;
import java.time.LocalDate;
import java.util.List;

public class MobileApp {
  private String name;

  private String version;

  private List<String> platform;

  private LocalDate releaseDate;

  private Url homepage;

The homepage data type is Url, and not java.net.URL as stated in documentation. Also, no import statement, so the generated class is invalid compile error.
Is this a bug on dgs, or my implementation is wrong?

Thanks

[berngp]

The codegen plugin requires an explicit Type Mapping. This caveat is missing in the extended scalar docs, will add it tomorrow.

[timpamungkas]

Added this

generateJava {
    typeMapping = ["Url"           : "java.net.URL",
                   "Time"          : "java.time.OffsetTime",
                   "DateTime"      : "java.time.OffsetDateTime",
                   "Date"          : "java.time.LocalDate",
                   "JSON"          : "com.fasterxml.jackson.databind.JsonNode",
                   "PositiveInt"   : "java.lang.Integer",
                   "NegativeFloat" : "java.lang.Double",
                   "NonPositiveInt": "java.lang.Integer"]
}

Thanks

[berngp]

The Spring Boot auto-configuration available in com.graphql-java:graphql-java-extended-scalars doesn't affect the CodeGen Plugin, its meant to be used by the server such that it understand how to SerDe types in the GraphQL request.

[timpamungkas]

Submitted PR for docs, thanks

[berngp]

Thanks for the PR, I reworded it a bit and merged. Result:

[berngp]

Just noticed you are using JsonNode to map it to the JSON scalar. Not sure that will work, probably you want to map JSON to just a "java.util.Map".