Skip to content

Commit

Permalink
Made AppInstanceEntity.cs properties nullable.
Browse files Browse the repository at this point in the history
  • Loading branch information
liammclennan committed Feb 24, 2022
1 parent 47faf0d commit 9ccfc0a
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 8 deletions.
18 changes: 11 additions & 7 deletions src/Seq.Api/Model/AppInstances/AppInstanceEntity.cs
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,8 @@
using Seq.Api.Model.Signals;
using Seq.Api.ResourceGroups;

#nullable enable

namespace Seq.Api.Model.AppInstances
{
/// <summary>
Expand Down Expand Up @@ -49,12 +51,12 @@ public AppInstanceEntity()
/// <summary>
/// The id of the <see cref="AppEntity"/> that this is an instance of.
/// </summary>
public string AppId { get; set; }
public string? AppId { get; set; }

/// <summary>
/// The user-friendly title of the app instance.
/// </summary>
public string Title { get; set; }
public string? Title { get; set; }

/// <summary>
/// Values for the settings exposed by the app.
Expand Down Expand Up @@ -95,7 +97,7 @@ public AppInstanceEntity()
/// The signal expression describing which events will be sent to the app; if <c>null</c>,
/// all events will reach the app.
/// </summary>
public SignalExpressionPart StreamedSignalExpression { get; set; }
public SignalExpressionPart? StreamedSignalExpression { get; set; }

/// <summary>
/// If a value is specified, events will be buffered to disk and sorted by timestamp-order
Expand Down Expand Up @@ -162,13 +164,15 @@ public AppInstanceEntity()
public bool? DisallowManualInput { get; set; }

/// <summary>
/// The name of the application.
/// The name of the app.
/// </summary>
public string AppName { get; set; }
[JsonProperty(DefaultValueHandling = DefaultValueHandling.Ignore)]
public string? AppName { get; set; }

/// <summary>
/// Is the application an input application?
/// If <c>true</c>, then the app is able to write events to the log.
/// </summary>
public bool IsInput { get; set; }
[JsonProperty(DefaultValueHandling = DefaultValueHandling.Ignore)]
public bool? IsInput { get; set; }
}
}
2 changes: 1 addition & 1 deletion src/Seq.Api/Model/Security/Permission.cs
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ public enum Permission
/// Access to administrative features of Seq, management of other users, app installation, backups.
/// </summary>
[Obsolete("The `Setup` permission has been replaced by `Project` and `System`.")]
Setup = 5,
Setup,

/// <summary>
/// Access to settings required for day-to-day operation of Seq, such as users, retention policies, API keys.
Expand Down

0 comments on commit 9ccfc0a

Please sign in to comment.