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

Array keypaths #141

Open
rstacruz opened this issue Aug 19, 2016 · 1 comment
Open

Array keypaths #141

rstacruz opened this issue Aug 19, 2016 · 1 comment

Comments

@rstacruz
Copy link
Contributor

It'd be nice to skip the string.split('.') and just allow array keypaths:

put(obj, ['user', 'name'], 'tjmenha') 

What do you think?

@tjmehta
Copy link
Owner

tjmehta commented Aug 25, 2016

All the keypath stuff is powered by keypather. It would be fine adding the functionality there, but adding this functionality in 101 complicates partial method arg checking..

Will have to think about this. I've seen this request before w/ keypather, but I always thought it was trivial to just use join('.'): put(obj, ['user', 'name'].join('.'), 'tjmehta')

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants