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

Rename "Windows Update tool" in VM/advanced #6628

Closed
olivierlambert opened this issue Jan 17, 2023 · 6 comments · Fixed by #6632
Closed

Rename "Windows Update tool" in VM/advanced #6628

olivierlambert opened this issue Jan 17, 2023 · 6 comments · Fixed by #6632

Comments

@olivierlambert
Copy link
Member

This is not clear, we need to rename it.

"Enable Citrix PV tools via Windows Update" is probably a lot better. Asking also @Fohdeesha and @Darkbeldin to get their opinion.

@Darkbeldin
Copy link
Contributor

Could be "Allow Windows update to install Citrix PV tools" but as discussed during the meeting we also need a warning if someone check the box saying this willbreak a VM with XCP tools

@olivierlambert
Copy link
Member Author

Windows update will not just install PV drivers (and not the management agent BTW), but also keep them up to date.

For the warning, you missed my other issue @Darkbeldin (I prefer to split problems into each issue). See #6627

@Fohdeesha
Copy link
Contributor

Instead of PV "tools" which implies the full management agent, should we name it something similar to "Download Citrix PV drivers via Windows Update" ?

maybe "Manage Citrix PV drivers via Windows Update" which better implies it will not only download drivers via windows update, but keep them up to date too? I like this best

@Darkbeldin
Copy link
Contributor

I think the windows update part of the process is the more important so it should start the message, so perhaps
"Allow Windows Update to install and manage Citrix PV driver"

@olivierlambert
Copy link
Member Author

Allow Windows Update to install and update Citrix PV drivers

It's very clear like this, but a bit long. We have something like 4 "medium/long" words tops. We can always add a "i" with a mouse hover on giving details, but then we need to choose what to display first and what to display in the mouse hover section.

@Fohdeesha
Copy link
Contributor

Hmm I can't think of anything much shorter that still conveys everything than "Manage Citrix PV drivers via Windows Update"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants