diff options
| author | Owen Jacobson <owen@grimoire.ca> | 2024-09-15 23:50:41 -0400 |
|---|---|---|
| committer | Owen Jacobson <owen@grimoire.ca> | 2024-09-16 11:03:22 -0400 |
| commit | 491cb3eb34d20140aed80dbb9edc39c4db5335d2 (patch) | |
| tree | e1e2e009f064dc6dfc8c98d2bf97d8d1f7b45615 /src/channel/routes.rs | |
| parent | 99b33023332393e46f5a661901b980b78e6fb133 (diff) | |
Consolidate most repository types into a repo module.
Having them contained in the individual endpoint groups conveyed an unintended sense that their intended scope was _only_ that endpoint group. It also made most repo-related import paths _quite_ long. This splits up the repos as follows:
* "General applicability" repos - those that are only loosely connected to a single task, and are likely to be shared between tasks - go in crate::repo.
* Specialized repos - those tightly connected to a specific task - go in the module for that task, under crate::PATH::repo.
In both cases, each repo goes in its own submodule, to make it easier to use the module name as a namespace.
Which category a repo goes in is a judgment call. `crate::channel::repo::broadcast` (formerly `channel::repo::messages`) is used outside of `crate::channel`, for example, but its main purpose is to support channel message broadcasts. It could arguably live under `crate::event::repo::channel`, but the resulting namespace is less legible to me.
Diffstat (limited to 'src/channel/routes.rs')
| -rw-r--r-- | src/channel/routes.rs | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/src/channel/routes.rs b/src/channel/routes.rs index 3c2353b..1379153 100644 --- a/src/channel/routes.rs +++ b/src/channel/routes.rs @@ -5,8 +5,12 @@ use axum::{ Router, }; -use super::repo::channels::Id as ChannelId; -use crate::{app::App, clock::RequestedAt, error::InternalError, login::repo::logins::Login}; +use crate::{ + app::App, + clock::RequestedAt, + error::InternalError, + repo::{channel, login::Login}, +}; pub fn router() -> Router<App> { Router::new() @@ -35,7 +39,7 @@ struct SendRequest { } async fn on_send( - Path(channel): Path<ChannelId>, + Path(channel): Path<channel::Id>, RequestedAt(sent_at): RequestedAt, State(app): State<App>, login: Login, |