Merge pull request 'proper log for registering guests' (#325) from staging into main
All checks were successful
CI/CD Pipeline / test (push) Successful in 10m14s
CI/CD Pipeline / deploy-staging (push) Has been skipped
CI/CD Pipeline / deploy-main (push) Successful in 5m13s

Reviewed-on: #325
This commit is contained in:
philipp 2024-03-29 21:46:58 +01:00
commit 1caced26d6
2 changed files with 21 additions and 11 deletions

View File

@ -11,7 +11,7 @@ impl UserTrip {
user: &User, user: &User,
trip_details: &TripDetails, trip_details: &TripDetails,
user_note: Option<String>, user_note: Option<String>,
) -> Result<(), UserTripError> { ) -> Result<String, UserTripError> {
if trip_details.is_full(db).await { if trip_details.is_full(db).await {
return Err(UserTripError::EventAlreadyFull); return Err(UserTripError::EventAlreadyFull);
} }
@ -81,7 +81,7 @@ impl UserTrip {
.await; .await;
} }
Ok(()) Ok(name_newly_registered_person)
} }
pub async fn delete( pub async fn delete(

View File

@ -63,7 +63,8 @@ async fn join(
}; };
match UserTrip::create(db, &user, &trip_details, user_note).await { match UserTrip::create(db, &user, &trip_details, user_note).await {
Ok(_) => { Ok(registered_user) => {
if registered_user == user.name {
Log::create( Log::create(
db, db,
format!( format!(
@ -72,6 +73,15 @@ async fn join(
), ),
) )
.await; .await;
}else{
Log::create(
db,
format!(
"User {} registered the guest '{}' for trip_details.id={}",
user.name, registered_user, trip_details_id
),
).await;
}
Flash::success(Redirect::to("/planned"), "Erfolgreich angemeldet!") Flash::success(Redirect::to("/planned"), "Erfolgreich angemeldet!")
} }
Err(UserTripError::EventAlreadyFull) => { Err(UserTripError::EventAlreadyFull) => {