From 4e3d5ccac99b24934c972e088cd7eb02bb95df06 Mon Sep 17 00:00:00 2001 From: Owen Jacobson Date: Tue, 17 Jun 2025 02:11:45 -0400 Subject: Handlers are _named operations_, which can be exposed via routes. Each domain module that exposes handlers does so through a `handlers` child module, ideally as a top-level symbol that can be plugged directly into Axum's `MethodRouter`. Modules could make exceptions to this - kill the doctrinaire inside yourself, after all - but none of the API modules that actually exist need such exceptions, and consistency is useful. The related details of request types, URL types, response types, errors, &c &c are then organized into modules under `handlers`, along with their respective tests. --- src/ui/routes/setup.rs | 43 ------------------------------------------- 1 file changed, 43 deletions(-) delete mode 100644 src/ui/routes/setup.rs (limited to 'src/ui/routes/setup.rs') diff --git a/src/ui/routes/setup.rs b/src/ui/routes/setup.rs deleted file mode 100644 index 649cc5f..0000000 --- a/src/ui/routes/setup.rs +++ /dev/null @@ -1,43 +0,0 @@ -pub mod get { - use axum::{ - extract::State, - response::{self, IntoResponse, Redirect}, - }; - - use crate::{ - app::App, - error::Internal, - ui::assets::{Asset, Assets}, - }; - - pub async fn handler(State(app): State) -> Result { - if app - .setup() - .completed() - .await - .map_err(Internal::from) - .map_err(Error::Internal)? - { - Err(Error::SetupCompleted) - } else { - Assets::index().map_err(Error::Internal) - } - } - - #[derive(Debug, thiserror::Error)] - pub enum Error { - #[error("setup already completed")] - SetupCompleted, - #[error("{0}")] - Internal(Internal), - } - - impl IntoResponse for Error { - fn into_response(self) -> response::Response { - match self { - Self::SetupCompleted => Redirect::to("/login").into_response(), - Self::Internal(error) => error.into_response(), - } - } - } -} -- cgit v1.2.3