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

bug with path & key #43

Closed
amark opened this issue Apr 29, 2015 · 2 comments
Closed

bug with path & key #43

amark opened this issue Apr 29, 2015 · 2 comments
Assignees
Milestone

Comments

@amark
Copy link
Owner

amark commented Apr 29, 2015

I THINK there is a potential bug:

if I do something like

gun.load('some/human')
  .path('girlfriend.cat')
  .key('some/magical/feline');

it SHOULD create a reference to the cat, however I think there is a bug, that it creates a reference to the immediate parent, aka the girlfriend, not the cat. This needs to be tested and fixed.

@metasean metasean added this to the 0.2.0 milestone May 20, 2015
@metasean metasean added InProgress and removed Queue labels May 20, 2015
@metasean metasean added the Core label Jun 2, 2015
@metasean
Copy link
Collaborator

metasean commented Jun 3, 2015

Need tests.

@amark amark added Pending and removed InProgress labels Jun 8, 2015
@amark
Copy link
Owner Author

amark commented Jul 8, 2015

Fixed in master, here. Closing!

@amark amark closed this as completed Jul 8, 2015
@amark amark removed the Pending label Jul 8, 2015
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