Skip to content

Unable to cast object of type 'Newtonsoft.Json.Linq.JProperty' to type 'Newtonsoft.Json.Linq.JObject'.#43

Closed
rasmuseeg wants to merge 2 commits intoRusticiSoftware:masterfrom
rasmuseeg:master
Closed

Unable to cast object of type 'Newtonsoft.Json.Linq.JProperty' to type 'Newtonsoft.Json.Linq.JObject'.#43
rasmuseeg wants to merge 2 commits intoRusticiSoftware:masterfrom
rasmuseeg:master

Conversation

@rasmuseeg
Copy link
Copy Markdown
Contributor

This fixes an issue with the older spec.

System.InvalidCastException: Unable to cast object of type 'Newtonsoft.Json.Linq.JProperty' to type 'Newtonsoft.Json.Linq.JObject'.
   at TinCan.ContextActivities..ctor(JObject jobj) in C:\Users\Administrator\Code\TinCan.NET\TinCan\ContextActivities.cs:line 38

@rasmuseeg
Copy link
Copy Markdown
Contributor Author

Any change this can be merged and released quickly?

@reedmclean
Copy link
Copy Markdown
Contributor

Rasmus,

This looks good. Can you add a test to cover this change? While you're adding the tests, could you also remove the empty merge commit?

I think your commit is valuable, so if you're unable to get around to this, I may go ahead and branch off of this and add the tests myself sometime in the next couple of days.

@reedmclean
Copy link
Copy Markdown
Contributor

Going to go ahead and close this PR. It was a part of #45, which was just closed. Thanks for the contribution!

@reedmclean reedmclean closed this Dec 17, 2018
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.

2 participants