partial update mutation

[Sed9yDataBank]

I am trying to update my position document which has 6 fields. I don't want to force updating all the fields whenever I call the update mutation.

@Mutation(() => Boolean, { nullable: false })
  async updatePosition(
    @Arg("positionId", type => ObjectIdScalar) positionId: ObjectId,
    @Arg("position", () => PositionInput, { nullable: true }) positionInput: PositionInput) {
      const position = await PositionModel.findOneAndUpdate({
          _id: positionId as any
      }, positionInput);

      if (position)
          return true;
      else return false;
  }

@InputType()
export class PositionInput implements Partial<Position> {
  @Field()
  positionName: string;

  @Field()
  positionOfficeLocation: string;

  @Field()
  positionDepartment: string;

  @Field()
  positionTotalNeeded: number;

  @Field()
  positionStatus: string;
}

What do I do if I only want to update the positionName, or if I am only want to update the positionStatus.
Do I need a different mutation for each field, I want to update?

[carlocorradini]

Set each field to nullable: true: if it's omitted no error from GraphQL is thrown.

Moreover, in the input class, each field that can be nullable/omitted should be set to optional using ?.

@InputType()
export class PositionUpdateInput implements Partial<Position> {
  @Field({ nullable: true })
  positionName?: string;

  @Field({ nullable: true })
  positionOfficeLocation?: string;

  @Field({ nullable: true })
  positionDepartment?: string;

  @Field({ nullable: true })
  positionTotalNeeded?: number;

  @Field({ nullable: true })
  positionStatus?: string;
}

@Mutation(() => Boolean, { nullable: false })
  async updatePosition(
    @Arg("positionId", type => ObjectIdScalar) positionId: ObjectId,
    @Arg("position", () => PositionUpdateInput) positionUpdateInput: PositionInput) {
      const position = await PositionModel.findOneAndUpdate({
          _id: positionId as any
      }, positionUpdateInput);

      if (position)
          return true;
      else return false;
  }

[Sed9yDataBank]

Thank you it works, didn't expect it to be this simple to tweak.

[Sed9yDataBank]

@carlocorradini could you please take a look at this? it's about a partial update in subdocument: #959