Represent channels and messages using a split "History" and "Snapshot" model.

This separates the code that figures out what happened to an entity from the code that represents it to a user, and makes it easier to compute a snapshot at a point in time (for things like bootstrap). It also makes the internal logic a bit easier to follow, since it's easier to tell whether you're working with a point in time or with the whole recorded history.

This hefty.

1 files changed, 6 insertions, 5 deletions

diff --git a/src/channel/routes.rs b/src/channel/routes.rs
index 5d8b61e..5bb1ee9 100644
--- a/src/channel/routes.rs
+++ b/src/channel/routes.rs
@@ -13,8 +13,9 @@ use crate::{
     channel::{self, Channel},
     clock::RequestedAt,
     error::Internal,
-    event::{app::EventsError, Sequence},
+    event::Sequence,
     login::Login,
+    message::app::Error as MessageError,
 };
 
 #[cfg(test)]
@@ -99,8 +100,8 @@ async fn on_send(
     login: Login,
     Json(request): Json<SendRequest>,
 ) -> Result<StatusCode, ErrorResponse> {
-    app.events()
-        .send(&login, &channel, &request.message, &sent_at)
+    app.messages()
+        .send(&channel, &login, &sent_at, &request.message)
         .await
         // Could impl `From` here, but it's more code and this is used once.
         .map_err(ErrorResponse)?;
@@ -109,13 +110,13 @@ async fn on_send(
 }
 
 #[derive(Debug)]
-struct ErrorResponse(EventsError);
+struct ErrorResponse(MessageError);
 
 impl IntoResponse for ErrorResponse {
     fn into_response(self) -> Response {
         let Self(error) = self;
         match error {
-            not_found @ EventsError::ChannelNotFound(_) => {
+            not_found @ MessageError::ChannelNotFound(_) => {
                 (StatusCode::NOT_FOUND, not_found.to_string()).into_response()
             }
             other => Internal::from(other).into_response(),