Closed rbuch closed 5 years ago
Looks good to me. All I wonder is: does it make more sense for the LogEntryData class to just be named LogEntry now? It seems like LogEntryData is really just a LogEntry now.
Agreed, I've renamed it now.
Agreed, I've renamed it now.