Open
Conversation
…ontend-integration
directories on a remote NFS in a unified view
directories on a remote NFS in a unified view
of the DME web application
…to frontend-integration
directories on a remote NFS in a unified view
…to frontend-integration
HPCDATAMGM-2083: Enhance the DME web application to display files and directories on a remote NFS in a unified view HPCDATAMGM-2090: Enable users to view directory size in the unified view of the DME web application
Error handling and loading overlay.
https://github.com/CBIIT/HPC_DME_APIs into frontend-integration
…to frontend-integration
Error handling and loading overlay.
Adding breadcrumb and icon to navigate to parent folder.
…ME_APIs into frontend-integration
directories on a remote NFS in a unified view Changing to obtain external path from query param.
…to frontend-integration
…to frontend-integration
…to frontend-integration
…to frontend-integration
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR introduces external archive browsing capabilities through comprehensive frontend integration. It adds a new Next.js-based frontend application alongside backend API endpoints for browsing external archives, with corresponding UI components and integration points.
- Frontend application for external archive browsing with React/Next.js components
- Backend API controllers for external archive operations and session management
- Configuration updates across environments to support frontend integration
Reviewed Changes
Copilot reviewed 48 out of 67 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| src/hpc-web/src/main/resources/templates/browse.html | Adds external archives UI integration to existing browse template |
| src/hpc-web/src/main/resources/static/result.html | Removes legacy static result page |
| src/hpc-web/src/main/resources/static/navbar.css | Removes legacy navbar styles |
| src/hpc-web/src/main/resources/static/css/style.css | Updates header positioning styles |
| src/hpc-web/src/main/resources/appconfigs/*.conf | Adds frontend server configuration across all environments |
| src/hpc-web/src/main/java/gov/nih/nci/hpc/web/util/HpcClientUtil.java | Adds external archive API client methods |
| src/hpc-web/src/main/java/gov/nih/nci/hpc/web/controller/*.java | Implements new API controllers for external archives and sessions |
| src/hpc-web/src/main/java/gov/nih/nci/hpc/web/HpcWebConfig.java | Adds CORS configuration for frontend integration |
| src/hpc-web/pom.xml | Integrates frontend build pipeline with Maven |
| src/hpc-web/hpc-web-frontend/* | Complete Next.js frontend application with components and configuration |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
src/hpc-web/src/main/java/gov/nih/nci/hpc/web/controller/HpcCalculateTotalSizeController.java
Outdated
Show resolved
Hide resolved
Comment on lines
+32
to
+34
| * <p> | ||
| * Unified browse controller to return user external archive to front end | ||
| * </p> |
There was a problem hiding this comment.
Class documentation describes it as 'Unified browse controller' but this is actually a calculate total size controller.
src/hpc-web/src/main/java/gov/nih/nci/hpc/web/util/HpcClientUtil.java
Outdated
Show resolved
Hide resolved
src/hpc-web/src/main/java/gov/nih/nci/hpc/web/util/HpcClientUtil.java
Outdated
Show resolved
Hide resolved
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…lculateTotalSizeController.java Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…il.java Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…il.java Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…to frontend-integration
from the unified view on the DME web application
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
External archive browsing capability front end changes.