plugins icon indicating copy to clipboard operation
plugins copied to clipboard

net/wireguard: Make wireguard widget configurable

Open patschi opened this issue 2 years ago • 5 comments

This is a first attempt to make the WireGuard widget configurable, as it was requested already.

Implements:

  1. Define to how much characters the public key should be truncated to. Or disable when using 0.
  2. Allows the user to hide specific columns

image

Closes #3212 Partially fixes #3211

patschi avatar Jan 23 '23 01:01 patschi

@fichtner Quick ping on this.

patschi avatar Jun 09 '23 20:06 patschi

@AdSchellevis I have seen you merged newer PR https://github.com/opnsense/plugins/pull/3349 now. Is there still any interest in merging this? This caused some merge conflicts now. Should I remove the public-key truncate feature?

patschi avatar Jun 28 '23 08:06 patschi

@patschi our time is a bit too limited to assess this further, if more feedback comes in, we can re-evaluate.

AdSchellevis avatar Jun 28 '23 10:06 AdSchellevis

At this point I'd rather refrain from adding config options to the widget to be honest.

fichtner avatar Jun 29 '23 07:06 fichtner

I can fully understand that time is valuable and often too short. But also to be frank, it's really very frustrating having invested hours to implement a useful feature for everyone, which got ignored about half a year and then rejected. My time is limited too, and it doesn't feel very good when PRs end up like this.

patschi avatar Jul 08 '23 22:07 patschi

New dashboard available with rewritten plugin, does not merge.

fichtner avatar Jan 10 '25 15:01 fichtner