Skip to content

Support for single member properties of ContextActivities #45

Merged
reedmclean merged 3 commits intomasterfrom
UnableToCast
Dec 17, 2018
Merged

Support for single member properties of ContextActivities #45
reedmclean merged 3 commits intomasterfrom
UnableToCast

Conversation

@reedmclean
Copy link
Copy Markdown
Contributor

This was branched off the main commit from #43 and added some tests to cover it.

@rasmuseeg
Copy link
Copy Markdown
Contributor

Great! Can this be released soon, it's blocking us from using the Nuget package?

@reedmclean
Copy link
Copy Markdown
Contributor Author

The goal is to get this reviewed and released early next week! I'll let you know if anything changes.

@brianjmiller brianjmiller self-requested a review December 17, 2018 17:57
Comment thread TinCan/ContextActivities.cs Outdated
parent = new List<Activity>();
foreach (JObject jactivity in jobj["parent"]) {
parent.Add((Activity)jactivity);
if(jobj["parent"].Type == JTokenType.Array)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Space needed before if.

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.

3 participants