fix: remove assert that might cause panic
This commit is contained in:
parent
e29d5d058d
commit
dc44d0ae4e
|
|
@ -19,16 +19,24 @@ use app_error::AppError;
|
||||||
|
|
||||||
#[inline]
|
#[inline]
|
||||||
pub async fn delete_from_workspace(pg_pool: &PgPool, workspace_id: &Uuid) -> Result<(), AppError> {
|
pub async fn delete_from_workspace(pg_pool: &PgPool, workspace_id: &Uuid) -> Result<(), AppError> {
|
||||||
let pg_row = sqlx::query!(
|
let res = sqlx::query!(
|
||||||
r#"
|
r#"
|
||||||
DELETE FROM public.af_workspace where workspace_id = $1
|
DELETE FROM public.af_workspace
|
||||||
|
WHERE workspace_id = $1
|
||||||
"#,
|
"#,
|
||||||
workspace_id
|
workspace_id
|
||||||
)
|
)
|
||||||
.execute(pg_pool)
|
.execute(pg_pool)
|
||||||
.await?;
|
.await?;
|
||||||
|
|
||||||
assert!(pg_row.rows_affected() == 1);
|
if res.rows_affected() != 1 {
|
||||||
|
tracing::error!(
|
||||||
|
"Failed to delete workspace, workspace_id: {}, rows_affected: {}",
|
||||||
|
workspace_id,
|
||||||
|
res.rows_affected()
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
Ok(())
|
Ok(())
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue