-
Notifications
You must be signed in to change notification settings - Fork 92
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(v3): FileSystemProvider implementation for Data Sets, USS and Jobs #2777
Conversation
Co-authored-by: Trae Yelovich <traeok@users.noreply.github.com> Signed-off-by: Timothy Johnson <timothy.johnson@broadcom.com>
Co-authored-by: Trae Yelovich <traeok@users.noreply.github.com> Co-authored-by: Fernando Rijo Cedeno <zFernand0@users.noreply.github.com> Signed-off-by: Timothy Johnson <timothy.johnson@broadcom.com>
Signed-off-by: Trae Yelovich <trae.yelovich@broadcom.com>
Signed-off-by: Trae Yelovich <trae.yelovich@broadcom.com>
…gement Signed-off-by: Trae Yelovich <trae.yelovich@broadcom.com>
Signed-off-by: Trae Yelovich <trae.yelovich@broadcom.com>
Signed-off-by: Trae Yelovich <trae.yelovich@broadcom.com>
Signed-off-by: Trae Yelovich <trae.yelovich@broadcom.com>
- added uploadBufferAsFile for FTP api - updated getContents to support "stream" (writing and reading from temp file) over FTP - updates to drag & drop for extender support Signed-off-by: Trae Yelovich <trae.yelovich@broadcom.com>
Signed-off-by: Trae Yelovich <trae.yelovich@broadcom.com>
Signed-off-by: Trae Yelovich <trae.yelovich@broadcom.com>
Signed-off-by: Trae Yelovich <trae.yelovich@broadcom.com>
…ptions in file context menu Signed-off-by: Trae Yelovich <trae.yelovich@broadcom.com>
…flict path Signed-off-by: Trae Yelovich <trae.yelovich@broadcom.com>
Signed-off-by: Trae Yelovich <trae.yelovich@broadcom.com>
…elocated doc on move Signed-off-by: Trae Yelovich <trae.yelovich@broadcom.com>
…es if removed Signed-off-by: Trae Yelovich <trae.yelovich@broadcom.com>
…ider Signed-off-by: Trae Yelovich <trae.yelovich@broadcom.com>
Signed-off-by: Trae Yelovich <trae.yelovich@broadcom.com>
Signed-off-by: Trae Yelovich <trae.yelovich@broadcom.com>
Signed-off-by: Trae Yelovich <trae.yelovich@broadcom.com>
Signed-off-by: Trae Yelovich <trae.yelovich@broadcom.com>
Signed-off-by: Trae Yelovich <trae.yelovich@broadcom.com>
Signed-off-by: Trae Yelovich <trae.yelovich@broadcom.com>
Signed-off-by: Trae Yelovich <trae.yelovich@broadcom.com>
Signed-off-by: Trae Yelovich <trae.yelovich@broadcom.com>
….TreeItem Signed-off-by: Trae Yelovich <trae.yelovich@broadcom.com>
Signed-off-by: Timothy Johnson <timothy.johnson@broadcom.com>
…mon logic Signed-off-by: Trae Yelovich <trae.yelovich@broadcom.com>
Signed-off-by: Trae Yelovich <trae.yelovich@broadcom.com>
Signed-off-by: Trae Yelovich <trae.yelovich@broadcom.com>
Signed-off-by: Trae Yelovich <trae.yelovich@broadcom.com>
Signed-off-by: Trae Yelovich <trae.yelovich@broadcom.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@zFernand0 Hmm... this must be specific to empty spool files, as although I am able to open all of my spool files, they all have contents. I can also open both empty data sets and empty USS files. Thanks for reporting this, I'll take a look 😅 Update: I think this should be resolved with |
Signed-off-by: Trae Yelovich <trae.yelovich@broadcom.com>
Signed-off-by: Trae Yelovich <trae.yelovich@broadcom.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When saving a USS file opened via an FTP profile, the save operation hangs for me.
Everything else LGTM - tested misc actions with both z/OSMF and FTP 😁
Signed-off-by: Trae Yelovich <trae.yelovich@broadcom.com>
Bug fix was implemented to resolve requested changes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Discussed with @traeok, the zFTP |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks @traeok!
Quality Gate failedFailed conditions |
With approvals will go ahead and merge, we can open and address issues for anything found dealing with trees in v3 separately. Thanks @traeok, @t1m0thyj, and @zFernand0 for all the hard work that went into the update and for all testers! This is HUGE! ❤️ |
Proposed changes
Check out the new page on ZE wiki for details on FSP infrastructure: https://github.com/zowe/zowe-explorer-vscode/wiki/FileSystemProvider
onResourceChanged
function to allow extenders to watch FileSystemProvider changesMisc. changes:
zowe.placeholderCommand
command so that we can disable items in the tree view 😋MockedProperty
class to clean up mocked properties defined usingObject.defineProperty
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 executed