For now with fixed options that are shared between most notification
events - later, once we have a notification registry, this should be
filled dynamically.

Signed-off-by: Lukas Wagner <l.wag...@proxmox.com>
---
 src/window/NotificationMatcherEdit.js | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/src/window/NotificationMatcherEdit.js 
b/src/window/NotificationMatcherEdit.js
index c6f0726..fb55e17 100644
--- a/src/window/NotificationMatcherEdit.js
+++ b/src/window/NotificationMatcherEdit.js
@@ -963,14 +963,23 @@ Ext.define('Proxmox.panel.NotificationMatchRuleSettings', 
{
        },
        {
            fieldLabel: gettext('Field'),
-           xtype: 'textfield',
+           xtype: 'proxmoxKVComboBox',
            isFormField: false,
            submitValue: false,
+           allowBlank: false,
+           editable: true,
+           displayField: 'key',
            bind: {
                hidden: '{!typeIsMatchField}',
                disabled: '{!typeIsMatchField}',
                value: '{matchFieldField}',
            },
+           // TODO: Once we have a 'notification registry', we should
+           // retrive those via an API call.
+           comboItems: [
+               ['type', ''],
+               ['hostname', ''],
+           ],
        },
        {
            fieldLabel: gettext('Value'),
-- 
2.39.2



_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to