Skip to content

JToken Merge Extensions#94

Closed
MrAntix wants to merge 3 commits intoJamesNK:masterfrom
MrAntix:master
Closed

JToken Merge Extensions#94
MrAntix wants to merge 3 commits intoJamesNK:masterfrom
MrAntix:master

Conversation

@MrAntix
Copy link
Copy Markdown

@MrAntix MrAntix commented May 31, 2013

Merge to create a new token which is the merge of two tokens
MergeInto to merge values into an existing token
Both overwrite the left with the right

MrAntix added 3 commits May 31, 2013 12:08
Merge to create a new token which is the merge of two tokens
MergeInto to merge values into an existing token
Both overwrite the left with the right
allow for concat or overwrite when merging array properties
@thorncp
Copy link
Copy Markdown

thorncp commented Feb 14, 2014

This feature would be perfect for my current needs, and all tests are passing on my machine. 👍

@MrAntix
Copy link
Copy Markdown
Author

MrAntix commented Feb 14, 2014

:)

thorncp added a commit to ResourceDataInc/Centroid that referenced this pull request Feb 14, 2014
Does not include any deep merge specific tests, only test the public API feature.

http://antix.co.uk/Blog/Merging-objects-using-JSON.NET

JamesNK/Newtonsoft.Json#94
thorncp added a commit to ResourceDataInc/Centroid that referenced this pull request Feb 14, 2014
Does not include any deep merge specific tests, only test the public API feature.

http://antix.co.uk/Blog/Merging-objects-using-JSON.NET

JamesNK/Newtonsoft.Json#94
thorncp added a commit to ResourceDataInc/Centroid that referenced this pull request Jun 26, 2014
thorncp added a commit to ResourceDataInc/Centroid that referenced this pull request Jun 26, 2014
@sahin
Copy link
Copy Markdown

sahin commented Jul 24, 2014

any updates on this PR?

@JamesNK
Copy link
Copy Markdown
Owner

JamesNK commented Aug 3, 2014

Added my own version of Merge to JObject/JArray/JConstructor.

@JamesNK JamesNK closed this Aug 3, 2014
@sahin
Copy link
Copy Markdown

sahin commented Aug 3, 2014

It is great. I would love to check deeper. Where is it in doc or which PR?

@MrAntix
Copy link
Copy Markdown
Author

MrAntix commented Aug 3, 2014

:(

@JamesNK
Copy link
Copy Markdown
Owner

JamesNK commented Aug 4, 2014

5fc7382

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.

4 participants