Skip to content

Commit e4491e8

Browse files
authored
Merge pull request #31 from contentstack/fix/CS-44119-deserialize-textnode
Fix:CS-44119 deserialize textnode
2 parents 768b780 + e677fa6 commit e4491e8

File tree

2 files changed

+17
-9
lines changed

2 files changed

+17
-9
lines changed

.github/workflows/nuget-publish.yml

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -12,32 +12,34 @@ jobs:
1212
- name: Checkout repository
1313
uses: actions/checkout@v1
1414
- name: Setup .NET Core @ Latest
15-
uses: actions/setup-dotnet@v1
15+
uses: actions/setup-dotnet@v3
1616
env:
17-
NUGET_AUTH_TOKEN: ${{ secrets.NUGET_AUTH_TOKEN }}
17+
NUGET_AUTH_TOKEN: ${{ secrets.NUGET_API_KEY }}
1818
- name: Build solution and generate NuGet package
1919
run: |
20-
cd contentstack-management-dotnet
2120
dotnet pack -c Release -o out
2221
2322
- name: Push generated package to GitHub registry
24-
run: dotnet nuget push ./contentstack-management-dotnet/out/*.nupkg --api-key $NUGET_AUTH_TOKEN --skip-duplicate --no-symbols true
23+
run: |
24+
cd out
25+
dotnet nuget push "contentstack.management.csharp.*.nupkg" --api-key ${{ secrets.NUGET_API_KEY }} --skip-duplicate --no-symbols true --source https://api.nuget.org/v3/index.json
2526
2627
publish-git:
2728
runs-on: windows-latest
2829
steps:
2930
- name: Checkout repository
3031
uses: actions/checkout@v1
3132
- name: Setup .NET Core @ Latest
32-
uses: actions/setup-dotnet@v1
33+
uses: actions/setup-dotnet@v3
3334
with:
3435
source-url: https://nuget.pkg.github.com/Contentstack/index.json
3536
env:
3637
NUGET_AUTH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
3738
- name: Build solution and generate NuGet package
3839
run: |
39-
cd contentstack-management-dotnet
4040
dotnet pack -c Release -o out
4141
4242
- name: Push generated package to GitHub registry
43-
run: dotnet nuget push ./contentstack-management-dotnet/out/*.nupkg --api-key $NUGET_AUTH_TOKEN --skip-duplicate --no-symbols true
43+
run: |
44+
cd out
45+
dotnet nuget push "contentstack.management.csharp.*.nupkg" --api-key ${{ secrets.NUGET_API_KEY }} --skip-duplicate --no-symbols true --source https://api.nuget.org/v3/index.json

Contentstack.Management.Core/Utils/TextNodeJsonConverter.cs

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,13 @@ public class TextNodeJsonConverter : JsonConverter<TextNode>
1010
{
1111
public override TextNode ReadJson(JsonReader reader, Type objectType, TextNode existingValue, bool hasExistingValue, JsonSerializer serializer)
1212
{
13-
throw new NotImplementedException();
13+
JObject jObject = JObject.Load(reader);
14+
15+
TextNode txtnode = new TextNode();
16+
17+
serializer.Populate(jObject.CreateReader(), txtnode);
18+
19+
return txtnode;
1420
}
1521

1622
public override void WriteJson(JsonWriter writer, TextNode value, JsonSerializer serializer)
@@ -33,7 +39,7 @@ public override void WriteJson(JsonWriter writer, TextNode value, JsonSerializer
3339
}
3440
writer.WriteEndArray();
3541
}
36-
// Write additional properties specific to TextNode
42+
3743
if (value.bold)
3844
{
3945
writer.WritePropertyName("bold");

0 commit comments

Comments
 (0)