frontend

backend/src/routes/documents.rs

@@ -133,7 +133,6 @@ impl From<Tag> for TagResponse {
 pub struct DocumentVersionResponse {
     pub id: Uuid,
     pub version_number: i32,
-    pub s3_key: String,
     pub size_bytes: i64,
     pub checksum: String,
     pub created_at: String,
@@ -175,7 +174,7 @@ pub struct DocumentAssetDetailResponse {
 }
 
 #[derive(Serialize, Clone, ToSchema)]
-pub struct DocumentCurrentVersionResponse {
+pub struct DocumentVersionDetailResponse {
     #[serde(flatten)]
     pub version: DocumentVersionResponse,
     #[serde(default, skip_serializing_if = "Vec::is_empty")]
@@ -209,7 +208,7 @@ pub struct DocumentResponse {
     #[serde(default, skip_serializing_if = "Vec::is_empty")]
     pub correspondents: Vec<DocumentCorrespondentResponse>,
     #[serde(skip_serializing_if = "Option::is_none")]
-    pub current_version: Option<DocumentCurrentVersionResponse>,
+    pub current_version: Option<DocumentVersionDetailResponse>,
 }
 #[derive(Serialize, ToSchema)]
 pub struct DocumentDetailResponse {
@@ -1145,6 +1144,73 @@ pub async fn get_document_asset(
     Ok(Json(to_asset_detail_response(asset, object_responses)))
 }
 
+pub async fn list_document_versions(
+    Path(document_id): Path<Uuid>,
+    TenantScopedConn {
+        mut conn,
+        tenant_id,
+        ..
+    }: TenantScopedConn,
+) -> AppResult<Json<Vec<DocumentVersionResponse>>> {
+    let document: Document = documents::table
+        .find(document_id)
+        .filter(documents::tenant_id.eq(tenant_id))
+        .first(&mut conn)?;
+
+    if document.deleted_at.is_some() {
+        return Err(AppError::not_found());
+    }
+
+    let versions: Vec<DocumentVersion> = document_versions::table
+        .filter(document_versions::document_id.eq(document_id))
+        .filter(document_versions::tenant_id.eq(tenant_id))
+        .order(document_versions::version_number.asc())
+        .load(&mut conn)?;
+
+    let versions: Vec<DocumentVersionResponse> =
+        versions.into_iter().map(to_version_response).collect();
+
+    Ok(Json(versions))
+}
+
+pub async fn get_document_version(
+    State(state): State<AppState>,
+    Path((document_id, version_id)): Path<(Uuid, Uuid)>,
+    TenantScopedConn {
+        mut conn,
+        tenant_id,
+        user_id,
+        ..
+    }: TenantScopedConn,
+) -> AppResult<Json<DocumentVersionDetailResponse>> {
+    let document: Document = documents::table
+        .find(document_id)
+        .filter(documents::tenant_id.eq(tenant_id))
+        .first(&mut conn)?;
+
+    if document.deleted_at.is_some() {
+        return Err(AppError::not_found());
+    }
+
+    let version: DocumentVersion = document_versions::table
+        .find(version_id)
+        .filter(document_versions::document_id.eq(document_id))
+        .filter(document_versions::tenant_id.eq(tenant_id))
+        .first(&mut conn)?;
+
+    drop(conn);
+
+    let assets = load_asset_responses(&state, tenant_id, version.id).await?;
+    let download_path = build_download_path(&state, &document, user_id)?;
+    let version_core = to_version_response(version);
+
+    Ok(Json(DocumentVersionDetailResponse {
+        version: version_core,
+        assets,
+        download_path,
+    }))
+}
+
 pub async fn download_document(
     State(state): State<AppState>,
     Path(document_id): Path<Uuid>,
@@ -2574,7 +2640,7 @@ pub(crate) fn to_document_response(
 ) -> AppResult<DocumentResponse> {
     let current_version = if let Some((version, assets)) = current_version {
         let download_path = build_download_path(state, &doc, user_id)?;
-        Some(DocumentCurrentVersionResponse {
+        Some(DocumentVersionDetailResponse {
             version,
             assets,
             download_path,

backend/src/routes/documents/asset_utils.rs

@@ -28,7 +28,6 @@ pub fn to_version_response(version: DocumentVersion) -> DocumentVersionResponse
     DocumentVersionResponse {
         id: version.id,
         version_number: version.version_number,
-        s3_key: version.s3_key,
         size_bytes: version.size_bytes,
         checksum: version.checksum,
         created_at: to_iso(version.created_at),

backend/src/routes/mod.rs

@@ -85,6 +85,11 @@ pub fn create_router(state: AppState) -> Router<()> {
             get(documents::list_document_assets).post(documents::request_document_assets),
         )
         .route("/:id/folder", patch(documents::move_document))
+        .route("/:id/versions", get(documents::list_document_versions))
+        .route(
+            "/:id/versions/:version_id",
+            get(documents::get_document_version),
+        )
         .route("/:id/restore", post(documents::restore_document))
         .route("/:id/tags", post(documents::assign_tags))
         .route("/:id/tags/:tag_id", delete(documents::remove_tag))