-
Notifications
You must be signed in to change notification settings - Fork 132
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
feat(nested keys): add ability to pass in keypath for object value #2
Conversation
f4ec62f
to
a8242dc
Compare
Current coverage is 100% (diff: 100%)@@ master #2 diff @@
===================================
Files 1 1
Lines 67 72 +5
Methods 0 0
Messages 0 0
Branches 0 0
===================================
+ Hits 67 72 +5
Misses 0 0
Partials 0 0
|
Oh, if you could keep that section in the roadmap and just replace the item with "ideas?" That'd be great. Also, would you like to add yourself to the contributors? Run |
a8242dc
to
cf00c47
Compare
@kentcdodds updated. thanks! |
LGTM |
Thank you! I just added you as a collaborator to the project. Want to merge your own pull request? |
Whoops, I forgot to add a maintainers file for lgtm.co/ If you rebase master and push that should fix it |
This allows you to pass in keypaths instead of just single keys. Makes things such as {keys: ['name.first']} possible. closes kentcdodds#1
cf00c47
to
9e00020
Compare
@kentcdodds thanks for the adding me! I rebased but it still doesn't seem to be going through. Let me know if there's anything i could do |
👍 |
LGTM |
Hmmm... Looks like something's wrong with lgtm.co :-( Maybe next PR it'll work better. Could be a caching issue. Anyway, thanks! And welcome to the team! |
This allows you to pass in keypaths instead of just single keys.
Makes things such as {keys: ['name.first']} possible.
closes #1