Optional arguments in hasura not working after migrating to hasura cloud

[lakhansamani]

Bug Details

With v1.3.3 following query used to work with an optional parameter

query getUser($isVerified: Boolean) {
  users(where: {
    isVerified: {
      _eq: $isVerified
    }
  }) {
    id
    email
    isVerified
  }
}

In the above query $isVerified is optional when it was not was passed it returned all the fields which is the expected behaviour.

After migrating to Hasura cloud version v2.0.0-alpha.2.cloud.2
The above query gives the following error when $isVerified is not passed.

{
  "errors": [
    {
      "extensions": {
        "path": "$.selectionSet.auth_users.args.where.isVerified._eq",
        "code": "validation-failed"
      },
      "message": "expected a boolean for type \"Boolean\", but found null"
    }
  ]
}

[tirumaraiselvan]

This is a breaking change in v2.0. From the release notes, behaviour changes section:

(Fixes behaviour of pathological dangerous case) The semantics of explicit null values in where filters have changed according to the discussion in issue #704: an explicit null value in a comparison input object will be treated as an error rather than resulting in the expression being evaluated to True. For instance: delete_users(where: {id: {_eq: $userId}}) { name } will yield an error if $userId is null instead of deleting all users.

[lakhansamani]

@tirumaraiselvan thanks for pointing out the changes.
As I see now one will have to create multiple queries for any such optional parameter, or is there a better alternative?

[lakhansamani]

Was able to resolve this, thanks @tirumaraiselvan

[bobymicroby]

Hi @lakhansamani , what was your approach to the problem ?

[lakhansamani]

@bobymicroby I used template literal here is an example

export const getQuery = (data) => {
   let query = `
    firstName: $firstName
    lastName: $lastName
    gender: $gender
    nationality: $nationality
    phone: $phone
  `;

  // for optional arguments
  if (data.isVerified) {
    query += `\n isVerified: $isVerified`
  }

  // final mutation
  const updateUser = gql`
     mutation updateProfile(
      $id: uuid!
      $firstName: String!
      $lastName: String!
      $gender: String!
      $nationality: String!
      $phone: String!
      $isVerified: Boolean
    ) {
      update_auth_users(
        where: { id: { _eq: $id } }
        _set: {
          ${query}
        }
      ) {
        affected_rows
      }
    }
  `;
   `
}

[bobymicroby]

@lakhansamani Thanks for the answer!

Since we are using TS & GraphQL generator for type safety, I will probably make $where: user_bool_exp! clause conditional instead of generating different queries.

Thanks again!