docs.nestjs.com icon indicating copy to clipboard operation
docs.nestjs.com copied to clipboard

docs(sql.md) update type of parameter of findOne method

Open ylcnfrht opened this issue 3 years ago • 0 comments

id converted to a number which is at findOne method because it shows error like (property) id?: number | FindOperator<number> Type 'string' is not assignable to type 'number | FindOperator<number>'.ts(2322)

also, id is the number at the given example https://github.com/nestjs/nest/blob/master/sample/05-sql-typeorm/src/users/users.service.ts#L26

PR Checklist

Please check if your PR fulfills the following requirements:

  • [ ] The commit message follows our guidelines: https://github.com/nestjs/docs.nestjs.com/blob/master/CONTRIBUTING.md

PR Type

What kind of change does this PR introduce?

  • [ ] Bugfix
  • [ ] Feature
  • [ ] Code style update (formatting, local variables)
  • [ ] Refactoring (no functional changes, no api changes)
  • [ ] Build related changes
  • [x] Docs
  • [ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Does this PR introduce a breaking change?

  • [ ] Yes
  • [x] No

Other information

ylcnfrht avatar Jul 22 '22 23:07 ylcnfrht