💡 Created PBIs from TODOs

src/Application/Common/Interfaces/IDateTime.cs

@@ -2,6 +2,5 @@
 
 public interface IDateTime
 {
- // TODO: Talk to Gordon about this - System Clock
  public DateTimeOffset Now => DateTimeOffset.UtcNow;
 }

src/Domain/Common/AuditableEntity.cs

@@ -3,7 +3,7 @@
 public abstract class AuditableEntity
 {
  public DateTimeOffset CreatedAt { get; set; }
- public string? CreatedBy { get; set; } // TODO: String as userId?
+ public string? CreatedBy { get; set; }
  public DateTimeOffset? UpdatedAt { get; set; }
- public string? UpdatedBy { get; set; } // TODO: String as userId?
+ public string? UpdatedBy { get; set; }
 }

src/Infrastructure/Persistence/Configuration/TodoItemConfiguration.cs

@@ -1,15 +1,11 @@
 using Microsoft.EntityFrameworkCore;
 using Microsoft.EntityFrameworkCore.Metadata.Builders;
-using Microsoft.EntityFrameworkCore.Storage.ValueConversion;
 using SSW.CleanArchitecture.Domain.Entities;
 
 namespace SSW.CleanArchitecture.Infrastructure.Persistence.Configuration;
 
 public class TodoItemConfiguration : IEntityTypeConfiguration<TodoItem>
 {
- // TODO: Rip out the common pieces that are from BaseEntity
- // virtual method, override 
- // Good marker to enforce that all entities have configuration defined via arch tests
  public void Configure(EntityTypeBuilder<TodoItem> builder)
  {
  builder.HasKey(t => t.Id);

src/WebApi/Features/TodoItemEndpoints.cs

@@ -20,9 +20,6 @@ public static void MapTodoItemEndpoints(this WebApplication app)
  .WithName("GetTodoItems")
  .ProducesGet<TodoItemDto[]>();
 
- // TODO: Investigate examples for swagger docs. i.e. better docs than:
- // myWeirdField: "string" vs myWeirdField: "this-silly-string"
-
  group
  .MapPost("/", (ISender sender, CreateTodoItemCommand command, CancellationToken ct) => sender.Send(command, ct))
  .WithName("CreateTodoItem")

src/WebApi/Filters/ExceptionFilter.cs

@@ -27,14 +27,11 @@ await context
 
  private static IResult HandleException(this HttpContext context, Exception exception)
  {
- // TODO: Null exception possibly?
-
  var type = exception.GetType();
 
  if (ExceptionHandlers.ContainsKey(type))
  return ExceptionHandlers[type].Invoke(context, exception);
 
- // TODO: Testing around unhandled exceptions
  return Results.Problem(statusCode: StatusCodes.Status500InternalServerError,
  type: "https://tools.ietf.org/html/rfc7231#section-6.6.1");
  }