-
Notifications
You must be signed in to change notification settings - Fork 112
Implements '*'(expand all) field selector. Closes: #147 #228
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
Conversation
smyrman
left a comment
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.
smyrman
left a comment
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 |
smyrman
left a comment
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.