Improve nullability of Reference.cs properties

[kylejuliandev]

Description

The nullability of ExternalId and Codename on Reference.cs is shown as not null in the Visual Studio IDE, it would be great to decorate these two properties (see below) with the nullability operator like on Id.

management-sdk-net/Kontent.Ai.Management/Models/Shared/Reference.cs

Line 24 in 5ab22f2

public string Codename { get; private set; }

management-sdk-net/Kontent.Ai.Management/Models/Shared/Reference.cs

Line 30 in 5ab22f2

public string ExternalId { get; private set; }

If I create a Reference instance with one of the provided factory methods (e.g. ByCodename, ByExternalId) the nullability of the remaining properties is not clear. In the sample below ExternalId is null although it's not documented as such.

using Kontent.Ai.Management.Models.Shared;

var reference = Reference.ByCodename("hello-world");

Console.WriteLine(reference.ExternalId);

Possibly long term it would be cool to enable nullable reference types (https://devblogs.microsoft.com/dotnet/try-out-nullable-reference-types/) to help make working with the Management SDK safer 😃

Many thanks!

[pokornyd]

hello and thank you for the submission. I'll review the proposed changes and update you on the resolution/PR, thank you for your patience :)