vscode-extension-for-zowe icon indicating copy to clipboard operation
vscode-extension-for-zowe copied to clipboard

Published 20 hours ago • verified publisher icon zowe

No UI indicator presented to end user after clicking "Pull from Mainframe"

Open traeok opened this issue 1 year ago • 8 comments

Describe the bug

There is no indication that the "Pull from Mainframe" action is ongoing once a user clicks it. We should show some sort of indication, such as a status bar indicator, to keep end users up to date.

To Reproduce

  1. Search for a data set
  2. Right-click on a data set member -> "Pull from Mainframe"
  3. Notice that no UI indicator appears.

Expected behavior

Users should see some indication that we are actively pulling data from the mainframe.

traeok avatar Sep 28 '23 17:09 traeok

Thank you for creating a bug report. We will investigate the bug and evaluate its impact on the product. If you haven't already, please ensure you have provided steps to reproduce the bug and as much context as possible.

github-actions[bot] avatar Sep 28 '23 17:09 github-actions[bot]

Can I work on the issue.

zaidmstrr avatar Nov 16 '23 06:11 zaidmstrr

Can I work on the issue.

@zaidmstrr Sure, I have assigned you to the issue, feel free to work on it 🙂

t1m0thyj avatar Nov 17 '23 15:11 t1m0thyj

Hi @t1m0thyj, can you please assign me this issue ? I'd love to work on it !

Ash-2k3 avatar Apr 14 '24 15:04 Ash-2k3

@Ash-2k3 Sure, I have assigned you the issue since it has not been worked on for 6 months, feel free to work on it 🙂

t1m0thyj avatar Apr 15 '24 13:04 t1m0thyj

Thanks @t1m0thyj, I have started working on it and I began with setting things up in my local machine, I am facing trouble with populating dummy data sets. I tried following this wiki https://github.com/zowe/zowe-explorer-vscode/wiki/Developer-Setup#test-profile-data. But it didn't really help me. I'd appreciate any help on this, Thanks!

Ash-2k3 avatar Apr 19 '24 06:04 Ash-2k3

@Ash-2k3 Currently the testProfileData is unused and the only integration tests we actively maintain are for Theia: https://github.com/zowe/zowe-explorer-vscode/wiki/Developer-Setup#run-zowe-explorer-regression-tests-for-theia

If you have made a fix locally and verified that unit tests are passing, then feel free to open a PR 🙂

t1m0thyj avatar Apr 19 '24 14:04 t1m0thyj

@t1m0thyj In that case, I think the wiki should be updated :). Anyways, I have opened a PR for this, but I am a bit skeptical about my approach since I couldn't really test my changes and also because I am new to this codebase, so I am not really used to coding pattern. But I do plan to get used to it overtime 😀. Thanks!

And also is there any issue which you would suggest me to work on after PR of this one gets merged ?

Ash-2k3 avatar Apr 22 '24 06:04 Ash-2k3