gorb/backend
5
0
Fork 1
Code Issues 14 Pull requests Projects Releases Packages 1 Wiki Activity Actions

wip/kick #36

Merged
radical merged 9 commits from wip/kick into main 2025-07-22 16:56:37 +00:00
Conversation 2 Commits 9 Files changed 4 +4 -5
3 changed files with 4 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit 31596c6bfe - Show all commits

3
src/api/v1/member/mod.rs → src/api/v1/members/mod.rs
View file

@ -10,9 +10,8 @@ use crate::{AppState, api::v1::auth::CurrentUser};
mod uuid; mod uuid;
pub fn router(app_state: Arc<AppState>) -> Router<Arc<AppState>> { pub fn router() -> Router<Arc<AppState>> {
Router::new() Router::new()
.route("/{uuid}", get(uuid::get)) .route("/{uuid}", get(uuid::get))
.route("/{uuid}", delete(uuid::delete)) .route("/{uuid}", delete(uuid::delete))
.layer(from_fn_with_state(app_state, CurrentUser::check_auth_layer))
} }

0
src/api/v1/member/uuid/mod.rs → src/api/v1/members/uuid/mod.rs
View file

6
src/api/v1/mod.rs
View file

@ -13,7 +13,7 @@ mod invites;
mod me; mod me;
mod stats; mod stats;
mod users; mod users;
mod member; mod members;
pub fn router(app_state: Arc<AppState>) -> Router<Arc<AppState>> { pub fn router(app_state: Arc<AppState>) -> Router<Arc<AppState>> {
let router_with_auth = Router::new() let router_with_auth = Router::new()
@ -29,7 +29,7 @@ pub fn router(app_state: Arc<AppState>) -> Router<Arc<AppState>> {
Router::new() Router::new()
.route("/stats", get(stats::res)) .route("/stats", get(stats::res))
.nest("/auth", auth::router(app_state.clone())) .nest("/auth", auth::router(app_state.clone()))
.nest("/channels", channels::router(app_state.clone())) .nest("/channels", channels::router(app_state))
.nest("/member", member::router(app_state)) .nest("/members", members::router())
radical marked this conversation as resolved Outdated
radical commented 2025-07-22 15:53:27 +00:00
Outdated
Copy link

this should be moved to the router_with_auth instead as all endpoints inside /members needs auth, also it should be /members plural and not /member singular. This also goes for the folder/module name

this should be moved to the router_with_auth instead as all endpoints inside /members needs auth, also it should be `/members` plural and not `/member` singular. This also goes for the folder/module name
baaboe commented 2025-07-22 15:59:15 +00:00
Outdated
Copy link

why plural, we are only accessing a single member

why plural, we are only accessing a single member
sauceyred commented 2025-07-22 16:06:49 +00:00
Outdated
Copy link

@baaboe wrote in #36 (comment):

why plural, we are only accessing a single member

Because it’s accessing a singular resource from the members resource collection. It’s a REST API thing.

@baaboe wrote in https://git.gorb.app/gorb/backend/pulls/36#issuecomment-831: > why plural, we are only accessing a single member Because it’s accessing a singular resource from the `members` resource *collection*. It’s a REST API thing.
radical commented 2025-07-22 16:21:53 +00:00
Outdated
Copy link

/members contains all members, even though you're only accessing one, think of it like a folder tree with /members containing all of the members of every guild out there but you only have access to a subset of them

`/members` contains *all* members, even though you're only accessing one, think of it like a folder tree with /members containing all of the members of every guild out there but you only have access to a subset of them
.merge(router_with_auth) .merge(router_with_auth)
} }