Skip to content
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

Issue TSO & MVS Commands Polish Work #1297

Closed
JillieBeanSim opened this issue May 3, 2021 · 3 comments
Closed

Issue TSO & MVS Commands Polish Work #1297

JillieBeanSim opened this issue May 3, 2021 · 3 comments
Labels

Comments

@JillieBeanSim
Copy link
Contributor

JillieBeanSim commented May 3, 2021

Polish item for issuing TSO and MVS commands, some may require a discussion.

What I would like to see:

  1. Separate console window for each instead of same title for both Zowe Command
  2. Revisit where the option is presented. I believe we should only present the option within the Data Sets view and not within all tree views. This may allow the validation of profiles icon to be present upon validation and remove some duplicate code I put in when first implemented this.
  3. Possibly separate the saved commands in the settings. At this time TSO & MVS commands are saved together under one section.
"Zowe Commands: History": {
    "persistence": true,
    "history": [],
    "fileHistory": [],
    "sessions": [],
    "searchHistory": [
      "PROFILE",
      "D T"
    ]
  },
  1. Investigate creation of TSO profiles or caching the account number while the VS Code session is in place as to not be prompted with every command issued if TSO profile isn't found.
  2. [FIXED] If escape when prompted for account number operation cancelled info message pops up but command text box also appears and needs to be escaped again. (bug)
  3. If escape when prompted for a command operation cancelled info message doesn't pop up...possible solution here
@JillieBeanSim JillieBeanSim added Technical Debt Includes Architecture, Code, Testing, Automation debt 21PI2 labels May 3, 2021
@JillieBeanSim JillieBeanSim self-assigned this May 3, 2021
@crawr crawr added the 21PI3 label Jul 22, 2021
@jellypuno jellypuno added the priority-medium Not functioning - next quarter if capacity permits label Jul 29, 2021
jellypuno added a commit that referenced this issue Sep 15, 2021
Issue #1297: Changing name of command window for TSO/MVS commands
jellypuno added a commit that referenced this issue Sep 15, 2021
Issue #1297: Fixed bug when cancelling TSO command
@jellypuno
Copy link
Contributor

Change to EPIC and break into smaller stories.
Can we add the creation of the TSO in ZE?

  • Maybe this is not needed since we are introducing the config
  • [Research] Check the schema and initialization of the TSO folder.
  • Currently, TSO Account number is prompted. You need to input the acct number every time you fire a command

@jellypuno jellypuno added Epic and removed Technical Debt Includes Architecture, Code, Testing, Automation debt priority-medium Not functioning - next quarter if capacity permits 21PI2 labels Oct 25, 2021
@JillieBeanSim JillieBeanSim added 22PI1 and removed 21PI4 labels Feb 7, 2022
@JillieBeanSim JillieBeanSim mentioned this issue Mar 31, 2022
55 tasks
@JillieBeanSim JillieBeanSim added 22PI2 and removed 22PI1 labels Apr 1, 2022
@zFernand0
Copy link
Member

A way we could polish a bit more the command handlers is to provide a common "handler" for any terminal-like functionality we have in ZE. (MVS, TSO, and possibly SSH). This command handler will keep an open connection and hook into the VSCode terminal. It could be influenced by extension settings (and should make it easier to save history)

@JillieBeanSim
Copy link
Contributor Author

JillieBeanSim commented Mar 9, 2023

that sounds awesome @zFernand0, and with removal of v1 profiles in v3 we will already have ssh & tso profiles easily accessible out the box upon config creation. I think v3 would be the best time to bring in the ssh support.

@JTonda JTonda closed this as completed Apr 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants