-
Notifications
You must be signed in to change notification settings - Fork 0
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
make root FeedDetails be same shape #62
Conversation
8d852cb
to
ea20caa
Compare
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 great! Just one request: can you remove the package-lock.json? And/or use .npmrc
with package-lock=false
?
I'll remove it and add it to the gitignore I've not used npmrc before... |
ea20caa
to
712b1f2
Compare
Fixed this. The commit history is weird. |
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. Feel free to Squash&Merge
when doing learning I expected the shape of the FeedDetails that come back from methods to be the same.
I noticed the
rootFeed
was lacking (only had seed, keys) and as a beginner this didn't help me understand the tree structure being builtThis PR brings that into alignment. I've based it on the README change I made where I presumed this was done