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

feat(nested keys): add ability to pass in keypath for object value #2

Merged
merged 1 commit into from
Aug 28, 2016

Conversation

conorhastings
Copy link
Collaborator

This allows you to pass in keypaths instead of just single keys.
Makes things such as {keys: ['name.first']} possible.

closes #1

@conorhastings conorhastings force-pushed the nested-key branch 5 times, most recently from f4ec62f to a8242dc Compare August 28, 2016 16:51
@codecov-io
Copy link

codecov-io commented Aug 28, 2016

Current coverage is 100% (diff: 100%)

Merging #2 into master will not change coverage

@@           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          

Powered by Codecov. Last update 6fe9c57...9e00020

@kentcdodds
Copy link
Owner

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 npm start add

@conorhastings
Copy link
Collaborator Author

@kentcdodds updated. thanks!

@kentcdodds
Copy link
Owner

LGTM

@kentcdodds
Copy link
Owner

Thank you! I just added you as a collaborator to the project. Want to merge your own pull request?

@kentcdodds
Copy link
Owner

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
@conorhastings
Copy link
Collaborator Author

@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

@kentcdodds
Copy link
Owner

:shipit:

@kentcdodds
Copy link
Owner

👍

@kentcdodds
Copy link
Owner

LGTM

@kentcdodds
Copy link
Owner

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!

@kentcdodds kentcdodds merged commit 6478bfc into kentcdodds:master Aug 28, 2016
@conorhastings conorhastings deleted the nested-key branch August 28, 2016 17:58
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.

support nested keys
3 participants