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

Research how we could optimize our call for multi-select functionality

Open jellypuno opened this issue 3 years ago • 3 comments

As a ZE developer, I would to find a way on how we could optimize our code to implement the multi-select feature without duplicating code.

jellypuno avatar Nov 23 '21 17:11 jellypuno

Thank you for creating a bug report. If you haven't already, please ensure you have provided steps to reproduce it and as much context as possible.

github-actions[bot] avatar Nov 23 '21 17:11 github-actions[bot]

@KutluOzel-b is research done? I know we have implemented a lot of actions in multi-select. FYI @zFernand0

JillieBeanSim avatar Oct 11 '22 17:10 JillieBeanSim

They way we are doing it know to call inline processes within loops to have batch processes and some of the functionality, that requires a specific batch process, is implemented as a separate batch process.

I think to ensure avoiding duplications and single point maintenance for achieving same functionality for inline or batch processes is the way we are using it now. Otherwise if the other way is chosen , which is batch processes for every functionality for every multi select, would open new branches to maintain codebase which is not a good practice.

Still some exceptions may be required in future depending on the functionality ( like all or none principle ) . For ex : delete all members of a pds if pds is completely deleted. ( I know our api does that but just for your imagination )

KutluOzel-b avatar Oct 12 '22 08:10 KutluOzel-b

closing as a duplicate to other multi-select issues

JillieBeanSim avatar Jan 25 '23 18:01 JillieBeanSim