-
Notifications
You must be signed in to change notification settings - Fork 19
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
Implement full Windows version/edition detection #4
Comments
If in the directory " |
My opinion - I would link to the compat list on the home page and not worry about having the app check. Patches to Windows (SP's and KB's) could easily add or remove support. For example, XP SP3 added support for some things not available in SP2. Forget what offhand, but trying to maintain the compatibility in the app is a never ending battle. Using sashaquert's approach of the RDP Wrapper requires the end users to have that installed on their system, so it is not 100% reliable. I know I wouldn't bother installing RDP Wrapper on a Windows Server machine with enough user CALs to cover my use case... plus not sure what licensing I may violate by using that. Just my 2 cents. |
Currently, you will be notified if the Windows edition detected is Home or Core.
This does not cover all incompatible Windows releases, such as Windows 7 or 8 Professional. I ignored these in the first instance because adding detection for Professional would exclude Windows 10 Professional (which is compatible). Compatibility list.
Need to implement detection of incompatible versions and editions in js/winver.js.
The text was updated successfully, but these errors were encountered: