-
-
Notifications
You must be signed in to change notification settings - Fork 32
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
feat: Move from jquery to react js #331
Conversation
b9cf8ca
to
785ebba
Compare
webpack/components/ProxmoxServer/components/NetworkInterface.js
Outdated
Show resolved
Hide resolved
webpack/components/ProxmoxContainer/ProxmoxContainerHardware.js
Outdated
Show resolved
Hide resolved
webpack/components/ProxmoxContainer/ProxmoxContainerHardware.js
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some minor nitpicks 😁
5e137e6
to
0426548
Compare
0426548
to
ca4d51c
Compare
ca4d51c
to
cc9f428
Compare
hv_evmcs: __( | ||
'Improve performance for nested virtualization. Only supported on Intel CPUs.' | ||
), | ||
aes: __('Activate AES instruction set for HW instruction'), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In the proxmox UI, it is actually:
"Activate AES instruction set for HW acceleration'"
'aes', | ||
]; | ||
|
||
const cpuFlagDescriptions = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Knit-pick: for some reason, the order in the UI differs from the order in the proxmox interface. But this is really, really minor.
No description provided.