graphene Default value for Argument's should be Undefined

Default value for Argument's should be Undefined

Open ewhauser opened this issue 4 years ago • 4 comments

When upgrading from 2 to 3, the default value of Argument's has changed from undefined to none which changes the schema. An example from test_relay_connection:

type Query {
  letters(before: String = null, after: String = null, first: Int = null, last: Int = null): LetterConnection
  connectionLetters(before: String = null, after: String = null, first: Int = null, last: Int = null): LetterConnection
  asyncLetters(before: String = null, after: String = null, first: Int = null, last: Int = null): LetterConnection
  node(
    """The ID of the object"""
    id: ID!
  ): Node
}

which should be:

type Query {
  letters(before: String, after: String, first: Int, last: Int): LetterConnection
  connectionLetters(before: String, after: String, first: Int, last: Int): LetterConnection
  asyncLetters(before: String, after: String, first: Int, last: Int): LetterConnection
  node(
    """The ID of the object"""
    id: ID!
  ): Node
}

Happy to submit a PR to fix if this is not intended behavior.