Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/nested query #42

Merged
merged 5 commits into from
Mar 22, 2019
Merged

Feature/nested query #42

merged 5 commits into from
Mar 22, 2019

Conversation

Diluka
Copy link
Contributor

@Diluka Diluka commented Mar 22, 2019

  1. nested field can be filtered by query
  2. page-info can be custom
  3. bug fix

1. support query on nested fields
2. add more swaggers
fix(core): dynamic create class for inject must change name
fix(swagger): add per_page alias
fix(tslint): suppress warnings and errors
1. page only when page param exists or return list like before
2. page-info response can be custom
3. `take` and `skip` distinguish `0` or `null` to decide if it is paging or listing
4. override `decidePagination` method can make it always paging or listing
@Diluka Diluka requested a review from michaelyali March 22, 2019 03:43
@michaelyali
Copy link
Member

Awesome! I'll review and merge by the EOD. Thanks!
Btw, are you in NestJs Discord channel? (https://discord.gg/jBcGvR)

@@ -223,6 +252,7 @@ export function setValidationPipe(crudOptions: CrudOptions, group: CrudValidate)
: undefined;
}

// FIXME Due to issue https://github.com/nestjs/nest/issues/1604, someone can't use global parser pipe without modify ParseIntPipe
export function setParseIntPipe() {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this function is deprecated.

@michaelyali michaelyali merged commit 033ad74 into nestjsx:3.0.0 Mar 22, 2019
@michaelyali michaelyali mentioned this pull request Mar 22, 2019
@Diluka Diluka deleted the feature/nested-query branch March 27, 2019 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants