Skip to content

Compose - #20

Merged
marcbachmann merged 2 commits into
janl:masterfrom
marcbachmann:compose
Sep 29, 2015
Merged

Compose#20
marcbachmann merged 2 commits into
janl:masterfrom
marcbachmann:compose

Conversation

@marcbachmann

@marcbachmannmarcbachmann commented May 27, 2015

Copy link
Copy Markdown
Collaborator
varjsonpointer=require('jsonpointer')varref=jsonpointer.compile('/a')ref.get(object)// returns object['a']ref.set(object,value)// sets object['a'] = value

Discussed in #8

marcbachmann
marcbachmann reviewed May 27, 2015
View reviewed changes
Comment threadjsonpointer.js Outdated

@marcbachmannmarcbachmannMay 27, 2015

Copy link
Copy Markdown
CollaboratorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe I should check value against undefined instead of a falsy value.

@marcbachmann

marcbachmann commented May 27, 2015

Copy link
Copy Markdown
CollaboratorAuthor

@hildjj still interested in this feature?

marcbachmann added a commit that referenced this pull request Sep 29, 2015
@marcbachmann
Merge pull request#20from marcbachmann/compose
dc1e2c6
jsonpointer.compile(pointer) feature
@marcbachmann
marcbachmann merged commit dc1e2c6 into janl:masterSep 29, 2015
@marcbachmann
marcbachmann deleted the compose branch September 29, 2015 23:09
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@marcbachmann