-
Notifications
You must be signed in to change notification settings - Fork 114
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
Implements '*'(expand all) field selector. Closes: #147 #228
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rs, any views?
c6adab2
to
37872a5
Compare
@smyrman This is whole revamped version of the functionality. There is a lot to digest, so take your time with the review. |
f901cce
to
3321a3a
Compare
Fixes `schema.Array` behaviour with `schema.Reference`. Adds projection validation to items within `schema.Array`.
3321a3a
to
6c8a96c
Compare
It may be more easy checking out my repo, and review with:
Because the patch on this PR, reverts part of my previous work on arrays of |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Still looks good to me.
3395ee4
to
7c3d20a
Compare
…place holder for other async resource (Reference,Connection)
710d028
to
bc68a53
Compare
No description provided.