tanishqgandhi1908 opened a new pull request, #7127:
URL: https://github.com/apache/texera/pull/7127

   ### What changes were proposed in this PR?
   
   Adds /dashboard/user/model/:mid, the read-only model detail page. Previously 
the Models list page linked to a route that didn't exist. The page is a 
view-only fork of dataset-detail.component.*: header with name, creation time, 
public/private and downloadable tags plus model-specific framework and format 
tags, quick-settings toggles, editable markdown description, version dropdown 
with per-version size, file tree, file preview, copy-path, and per-file / 
per-version-zip download. It reuses user-dataset-version-filetree, 
markdown-description and size-formatter.util.ts rather than duplicating them.
   
   user-dataset-file-renderer gains a single @Input() resourceType, defaulting 
to "dataset", which branches only its blob fetch — so every existing call site 
is unchanged. ModelService gains the version/file-tree/presign/zip methods, 
DownloadService gains the model download wrappers, and a ModelVersion interface 
is added.
   
   
   ### Any related issues, documentation, discussions?
   
   Closes #6499 (Add model management UI). 
   Umbrella: #6494.
   
   Stacked on fix/model-file-tree-resource-prefix
   
   ### How was this PR tested?
   
   New model-detail.component.spec.ts with 25 cases covering route-param 
handling, metadata population, version selection and file-tree loading, 
download gating across owner / public / no-access, both toggles including 
failure-revert, description optimistic-update-and-revert, and the empty-version 
and empty-tree paths. Plus 7 cases in model.service.spec.ts, 4 in 
download.service.spec.ts, and 1 model-branch case in the renderer spec.
   
   The renderer's 18 pre-existing tests and all 63 dataset-detail tests pass 
unmodified, which is the evidence the shared seam is behaviour-neutral for 
datasets. 199 tests pass across the six affected spec files; tsc --noEmit, 
eslint and prettier are clean.
   
   cd frontend && npx ng test --watch=false --include 
src/app/dashboard/component/user/user-model/user-model-explorer/model-detail.component.spec.ts
   
   Manually verified against a local stack: card click reaches the page, tags 
render correctly, the visibility and downloadable toggles persist, description 
editing works, and a model with no versions shows the empty state. File preview 
and version switching will be exercised once the create-model/version UI lands 
in the next PR.
   
   <img width="1199" height="652" alt="Screenshot 2026-07-30 at 10 21 14 AM" 
src="https://github.com/user-attachments/assets/bd83c372-0ba3-4e82-9608-9e6cec930875";
 />
   
   <img width="1231" height="687" alt="Screenshot 2026-07-30 at 10 19 32 AM" 
src="https://github.com/user-attachments/assets/7770c1cf-df42-4418-8393-8888865ee632";
 />
   
   ### Was this PR authored or co-authored using generative AI tooling?
   
   Generated-by: Claude Code (Claude Opus 5)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to