fix: move 401 check from readFile into stat - Githubissues

zowe / zowe-explorer-vscode

Visual Studio Code Extension for Zowe, which lets users interact with z/OS Data Sets, Unix System Services, and Jobs on a remote mainframe instance. Powered by Zowe SDKs.

Eclipse Public License 2.0

173 stars 92 forks source link

fix: move 401 check from readFile into stat #3320

Closed traeok closed 1 week ago

traeok commented 1 week ago

Proposed changes

Fixes issue where for previously accessed USS or data set files, the 401 prompt in editor does not appear

Release Notes

Milestone: 3.1.0 does not need ported to 3.0.3, changes already in #3318

Changelog: No changelog since changes aren't published

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (non-breaking change which adds or improves functionality)
[ ] Breaking change (a change that would cause existing functionality to not work as expected)
[ ] Documentation (Markdown, README updates)
[ ] Other (please specify above in "Proposed changes" section)

Checklist

General

[x] I have read the CONTRIBUTOR GUIDANCE wiki
[ ] All PR dependencies have been merged and published (if applicable)
[ ] A GIF or screenshot is included in the PR for visual changes
[ ] The pre-publish command has been executed:
- v2 and below: yarn workspace vscode-extension-for-zowe vscode:prepublish
- v3: pnpm --filter vscode-extension-for-zowe vscode:prepublish

Code coverage

[x] There is coverage for the code that I have added
[x] I have added new test cases and they are passing
[x] I have manually tested the changes

Deployment

[ ] I have added developer documentation (if applicable)
[ ] Documentation should be added to Zowe Docs
- If you're an outside contributor, please post in the #zowe-doc Slack channel to coordinate documentation.
- Otherwise, please check with the rest of the squad about any needed documentation before merging.
[ ] These changes may need ported to the appropriate branches (list here):

Further comments

github-actions[bot] commented 1 week ago

📅 Suggested merge-by date: 11/28/2024