[PATCH pve-manager] ui: add online help link for prefix lists and route maps

[PATCH pve-manager] ui: add online help link for prefix lists and route maps

[Christian Ebner]

These have several config options which are worth to be easily looked
up by directly providing the link to the docs.

Signed-off-by: Christian Ebner <c.ebner@proxmox.com>
---
 www/manager6/sdn/PrefixListPanel.js | 2 ++
 www/manager6/sdn/RouteMapPanel.js   | 2 ++
 2 files changed, 4 insertions(+)

diff --git a/www/manager6/sdn/PrefixListPanel.js b/www/manager6/sdn/PrefixListPanel.js
index d8ce3db8..d678356c 100644
--- a/www/manager6/sdn/PrefixListPanel.js
+++ b/www/manager6/sdn/PrefixListPanel.js
@@ -45,6 +45,8 @@ Ext.define('PVE.sdn.EditPrefixListEntryWindow', {
     extend: 'Proxmox.window.Edit',
     mixins: ['Proxmox.Mixin.CBind'],
 
+    onlineHelp: 'pvesdn_config_prefix_lists',
+
     // TRANSLATORS: Refers to an FRR prefix list, some languages may prefer to
     // keep "prefix list" as-is:
     // https://docs.frrouting.org/en/latest/filter.html#ip-prefix-list
diff --git a/www/manager6/sdn/RouteMapPanel.js b/www/manager6/sdn/RouteMapPanel.js
index 46f243fb..25630d8c 100644
--- a/www/manager6/sdn/RouteMapPanel.js
+++ b/www/manager6/sdn/RouteMapPanel.js
@@ -709,6 +709,8 @@ Ext.define('PVE.sdn.EditRouteMapEntryWindow', {
     // https://docs.frrouting.org/en/latest/routemap.html
     subject: gettext('Route Map Entry'),
 
+    onlineHelp: 'pvesdn_config_route_maps',
+
     initComponent: function () {
         let me = this;
         me.method = me.isCreate ? 'POST' : 'PUT';
-- 
2.47.3

Re: [PATCH pve-manager] ui: add online help link for prefix lists and route maps

[Shannon Sterz]

On Tue May 19, 2026 at 2:29 PM CEST, Christian Ebner wrote:
> These have several config options which are worth to be easily looked
> up by directly providing the link to the docs.
>
> Signed-off-by: Christian Ebner <c.ebner@proxmox.com>
> ---
>  www/manager6/sdn/PrefixListPanel.js | 2 ++
>  www/manager6/sdn/RouteMapPanel.js   | 2 ++
>  2 files changed, 4 insertions(+)
>
> diff --git a/www/manager6/sdn/PrefixListPanel.js b/www/manager6/sdn/PrefixListPanel.js
> index d8ce3db8..d678356c 100644
> --- a/www/manager6/sdn/PrefixListPanel.js
> +++ b/www/manager6/sdn/PrefixListPanel.js
> @@ -45,6 +45,8 @@ Ext.define('PVE.sdn.EditPrefixListEntryWindow', {
>      extend: 'Proxmox.window.Edit',
>      mixins: ['Proxmox.Mixin.CBind'],
>
> +    onlineHelp: 'pvesdn_config_prefix_lists',
> +
>      // TRANSLATORS: Refers to an FRR prefix list, some languages may prefer to
>      // keep "prefix list" as-is:
>      // https://docs.frrouting.org/en/latest/filter.html#ip-prefix-list
> diff --git a/www/manager6/sdn/RouteMapPanel.js b/www/manager6/sdn/RouteMapPanel.js
> index 46f243fb..25630d8c 100644
> --- a/www/manager6/sdn/RouteMapPanel.js
> +++ b/www/manager6/sdn/RouteMapPanel.js
> @@ -709,6 +709,8 @@ Ext.define('PVE.sdn.EditRouteMapEntryWindow', {
>      // https://docs.frrouting.org/en/latest/routemap.html
>      subject: gettext('Route Map Entry'),
>
> +    onlineHelp: 'pvesdn_config_route_maps',
> +
>      initComponent: function () {
>          let me = this;
>          me.method = me.isCreate ? 'POST' : 'PUT';

ah sorry somehow missed that and send a similar patch now [1]. however,
yours add help buttons to the edit/add dialogs, whereas mine affects the
top right "Help" button in the panels. imo it could still make sense to
apply both here.

not applying mine would mean that those "Help" buttons would essentially
become a second "Documentation" button.

[1]: https://lore.proxmox.com/pve-devel/20260519150415.334856-1-s.sterz@proxmox.com/T/#m59a50d54e8b5df41a3dd9ea0ecbe73ba9b9dc41a