Thanks to visit codestin.com
Credit goes to github.com

Skip to content

feat(nested): support nested GQL #52

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.

Sign up for GitHub

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

Jump to bottom
Merged
thanh-nguyen-dang merged 5 commits into from
Feb 10, 2020
Merged

feat(nested): support nested GQL #52

thanh-nguyen-dang merged 5 commits into from
Feb 10, 2020

Conversation

@thanh-nguyen-dang
Copy link
Contributor

@thanh-nguyen-dang thanh-nguyen-dang commented Aug 28, 2019 •
edited
Loading

Add nested fields and filter on those nested field.

New Features

  • Nested field with filter availability

@thanh-nguyen-dang thanh-nguyen-dang force-pushed the feat/nested branch 3 times, most recently from d89d7de to 69fc3b9 Compare August 28, 2019 20:49
@thanh-nguyen-dang thanh-nguyen-dang force-pushed the feat/nested branch 5 times, most recently from 199140f to 0fa7daa Compare October 2, 2019 07:09
@thanh-nguyen-dang thanh-nguyen-dang force-pushed the feat/nested branch 9 times, most recently from 2d1801e to ceef09a Compare November 1, 2019 19:18
ZakirG
ZakirG reviewed Nov 1, 2019
View reviewed changes
qingyashu
qingyashu reviewed Nov 4, 2019
View reviewed changes
qingyashu
qingyashu reviewed Nov 4, 2019
View reviewed changes
m0nhawk
m0nhawk reviewed Nov 6, 2019
View reviewed changes
m0nhawk
m0nhawk reviewed Nov 6, 2019
View reviewed changes
m0nhawk
m0nhawk previously approved these changes Nov 6, 2019
View reviewed changes
qingyashu
qingyashu reviewed Nov 11, 2019
View reviewed changes
qingyashu
qingyashu reviewed Nov 14, 2019
View reviewed changes
qingyashu
qingyashu reviewed Nov 14, 2019
View reviewed changes
@lgtm-com
Copy link

lgtm-com bot commented Jan 2, 2020

This pull request fixes 1 alert when merging 57b4a14 into 29a968f - view on LGTM.com

fixed alerts:

  • 1 for Superfluous trailing arguments

@lgtm-com
Copy link

lgtm-com bot commented Jan 3, 2020

This pull request fixes 1 alert when merging 48e22ca into 29a968f - view on LGTM.com

fixed alerts:

  • 1 for Superfluous trailing arguments

@lgtm-com
Copy link

lgtm-com bot commented Jan 3, 2020

This pull request fixes 1 alert when merging a523d3d into 29a968f - view on LGTM.com

fixed alerts:

  • 1 for Superfluous trailing arguments

@lgtm-com
Copy link

lgtm-com bot commented Jan 3, 2020

This pull request fixes 1 alert when merging 9874513 into 29a968f - view on LGTM.com

fixed alerts:

  • 1 for Superfluous trailing arguments

qingyashu
qingyashu reviewed Jan 9, 2020
View reviewed changes
@qingyashu
Copy link
Contributor

qingyashu commented Jan 9, 2020

I run npm i under this branch and got npm error when installing the json-schema-faker package (package's master branch), so I just use this pinned version from npm website https://www.npmjs.com/package/json-schema-faker/v/0.5.0-rc15, but seems generated data got crazy mappings:
image
not sure it's the package's problem or script's problem

@qingyashu
Copy link
Contributor

qingyashu commented Jan 22, 2020

fake data issue fixed using "json-schema-faker": "0.5.0-rc23", version

qingyashu
qingyashu reviewed Feb 5, 2020
View reviewed changes
qingyashu
qingyashu reviewed Feb 5, 2020
View reviewed changes
qingyashu
qingyashu reviewed Feb 5, 2020
View reviewed changes
qingyashu
qingyashu reviewed Feb 5, 2020
View reviewed changes
@lgtm-com
Copy link

lgtm-com bot commented Feb 7, 2020

This pull request fixes 1 alert when merging 07035cf into f970809 - view on LGTM.com

fixed alerts:

  • 1 for Superfluous trailing arguments

@lgtm-com
Copy link

lgtm-com bot commented Feb 7, 2020

This pull request fixes 1 alert when merging 9fc6551 into f970809 - view on LGTM.com

fixed alerts:

  • 1 for Superfluous trailing arguments

@lgtm-com
Copy link

lgtm-com bot commented Feb 7, 2020

This pull request fixes 1 alert when merging 583edbf into f970809 - view on LGTM.com

fixed alerts:

  • 1 for Superfluous trailing arguments

qingyashu
qingyashu reviewed Feb 7, 2020
View reviewed changes
@lgtm-com
Copy link

lgtm-com bot commented Feb 10, 2020

This pull request fixes 1 alert when merging 00c614a into 47c8406 - view on LGTM.com

fixed alerts:

  • 1 for Superfluous trailing arguments

@thanh-nguyen-dang thanh-nguyen-dang merged commit 2b2c520 into master Feb 10, 2020
@thanh-nguyen-dang thanh-nguyen-dang deleted the feat/nested branch February 10, 2020 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mfshao mfshao mfshao left review comments

@m0nhawk m0nhawk m0nhawk left review comments

+2 more reviewers

@ZakirG ZakirG ZakirG left review comments

@qingyashu qingyashu qingyashu approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@thanh-nguyen-dang @qingyashu @mfshao @ZakirG @m0nhawk