Closed likhithanimma1 closed 5 months ago
Attention: Patch coverage is 92.59259%
with 2 lines
in your changes are missing coverage. Please review.
Project coverage is 93.32%. Comparing base (
8cd2da0
) to head (ff874a1
).
Files | Patch % | Lines |
---|---|---|
packages/zowe-explorer/src/uss/actions.ts | 90.47% | 2 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
I was looking around and found #1956 - I think this PR will implement the request from that issue 👍 Maybe we should reference that issue in the changelog entry as well?
Thanks for working on this enhancement @likhithanimma1!
I noticed that two status bar messages appear at the same time as the upload:
- "Uploading file to USS tree"
- "Upload action was cancelled"
After the "Uploading file..." message disappears, it seems the file was not successfully uploaded as I cannot fetch the file in the destination folder afterward:
Addressed the behaviour!
Thanks for addressing the withProgress and tree refresh, this is looking good. I did experience the same issue seen as Trae with the filename missing from delete dialog but only with the upload of binary, same scenario works fine with regular upload.
Thanks for addressing the withProgress and tree refresh, this is looking good. I did experience the same issue seen as Trae with the filename missing from delete dialog but only with the upload of binary, same scenario works fine with regular upload.
Addressed deletion of files after uploading @JillieBeanSim
Issues
0 New issues
0 Accepted issues
Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code
Proposed changes
Provide users with the option to upload binary files by implementing a "Upload Files (Binary)" right-click option in the USS tree.
Release Notes
Milestone:
Changelog:
Types of changes
What types of changes does your code introduce to Zowe Explorer? Put an
x
in the boxes that applyChecklist
Put an
x
in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This checklist will be used as reference for both the contributor and the revieweryarn workspace vscode-extension-for-zowe vscode:prepublish
has been executedFurther comments