Some code style remarks
Code looks clean overall but I want to point out a few things:
- why not use an uninhabited type like an empty enum (enum with no cases) in RBConstants.swift to prevent instantiations of the
- in RBManager.swift which seems to describe a singleton, why not clearly add a private initializer to make sure that only one instance will be created ?
- you seem to retrieve timestamps in a few places (in RBManager.swift or Message/MessageManager.swift) using a helper
Datewhich returns an
Int, so why those are wrapped again in Int values like this: