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.

Sign up for GitHub

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

Jump to bottom

Support the Delete Profile option for team config files #3037

Open
Tracked by #3166
adam-wolfe opened this issue Aug 1, 2024 · 6 comments · May be fixed by #3251
Open
Tracked by #3166

Support the Delete Profile option for team config files #3037

adam-wolfe opened this issue Aug 1, 2024 · 6 comments · May be fixed by #3251
Assignees
@t1m0thyj
Labels
enhancement New feature or request priority-medium Not functioning - next quarter if capacity permits Research Needed
Milestone
v3.1.0

Comments

@adam-wolfe
Copy link
Contributor

Is your feature request related to a problem? Please describe.

In V3, users will not be using V1 profiles. With a team config, the "Delete Profile" option in the profile management menu is functionally identical to the "Edit Profile" option.

Describe the solution you'd like

Remove the "Delete Profile" option from the profile management menu.

It might also be clearer to change "Edit Profile" to "Edit Configuration File" or "Edit Profiles" or something else that indicates that Zowe Explorer will open the team config in an editor tab.

Describe alternatives you've considered

Additional context
@adam-wolfe adam-wolfe added the enhancement New feature or request label Aug 1, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented Aug 1, 2024

Thank you for raising this enhancement request.
The community has 90 days to vote on it.
If the enhancement receives at least 10 upvotes, it is added to our development backlog.
If it receives fewer votes, the issue is closed.

@t1m0thyj
Copy link
Member

t1m0thyj commented Aug 1, 2024

As an alternative, should we consider keeping the Delete Profile option and making it work with team config files?

@t1m0thyj
Copy link
Member

t1m0thyj commented Aug 1, 2024 •
edited
Loading

Created a quick PoC of how the option could work if we decide to keep it: feat/delete-v2-profiles
delete-profiles-v2

@adam-wolfe
Copy link
Contributor Author

As an alternative, should we consider keeping the Delete Profile option and making it work with team config files?

I think that would be ideal.

@JTonda
Copy link

JTonda commented Aug 6, 2024

Add a warning to prompt for confirmation before deleting that mentions if the profile is a default and if the profile contains child profiles.

@JTonda JTonda added Research Needed priority-high Production outage - this quarter or at least next quarter labels Aug 6, 2024
@zFernand0 zFernand0 mentioned this issue Aug 6, 2024
Open
@zFernand0
Copy link
Member

Note: The research needed label was added because we would like to determine if it is worth making a change in the underlying Imperative.ProfileInfo APIs in order to ease the burden on extenders and API users when managing profile.

@t1m0thyj t1m0thyj self-assigned this Aug 23, 2024
@JTonda JTonda added this to the v3.1.0 milestone Sep 12, 2024
@JillieBeanSim JillieBeanSim added priority-medium Not functioning - next quarter if capacity permits and removed priority-high Production outage - this quarter or at least next quarter labels Sep 17, 2024
@t1m0thyj t1m0thyj changed the title Remove the Delete Profile option from the profile management menu in V3 Support the Delete Profile option for team config files Sep 17, 2024
@JillieBeanSim JillieBeanSim mentioned this issue Oct 2, 2024
Open
1 task
@t1m0thyj t1m0thyj linked a pull request Oct 17, 2024 that will close this issue
WIP Support the Delete Profile option for team config files #3251
Draft
15 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@t1m0thyj t1m0thyj

Labels
enhancement New feature or request priority-medium Not functioning - next quarter if capacity permits Research Needed
Projects
Zowe Explorer for VS Code
Status: Medium Priority
Milestone
v3.1.0
Development

Successfully merging a pull request may close this issue.

WIP Support the Delete Profile option for team config files zowe/zowe-explorer-vscode
5 participants
@t1m0thyj @JTonda @zFernand0 @JillieBeanSim @adam-wolfe