Skip to content
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

v2 image testing results #206

Open
pasztorl opened this issue Jan 6, 2024 · 6 comments
Open

v2 image testing results #206

pasztorl opened this issue Jan 6, 2024 · 6 comments

Comments

@pasztorl
Copy link

pasztorl commented Jan 6, 2024

HI,

I'm testing the v2 image.

There is one wg interface configured in server mode.

  1. I set up an ip cidr in Edit interface -> Interface -> "IP Addresses" it not configured to the wg interface.

  2. So I set up a post-up hook to run ip command to assign IP address to the wg interface, but it seems the command not runned when I click to save.

  3. Int the Interface status for wg interface not shows the interface's IP address

  4. When I set up a peer with allowedIPs option the site saves it, but not set it up to the wg interface.

In the log I haven't seen any errors.
This should be works, but I miss something? How can I debug it?

@pasztorl
Copy link
Author

pasztorl commented Jan 6, 2024

Edit interface -> Peer defaults not saves the IP Networks, Allowed IP Addresses, DNS Server, DNS Search Domains fields

@HPPinata
Copy link
Contributor

HPPinata commented Jan 8, 2024

Are you hitting space after typing them in? If I remember correctly the entrys in those fields should be highlighted in blue when they actually apply. This only happens if you hit space bar after typing them in (even if it's just one entry). Otherwise they are still "saved" in the sense that you will see them after reloading and opening the page again but they won't apply.

@pasztorl
Copy link
Author

pasztorl commented Jan 8, 2024

ahh that was the problem, thanks! continuing the test..

@HPPinata
Copy link
Contributor

HPPinata commented Jan 8, 2024

Potential improvement:
@h44z maybe there should be a help text or a warning when saving if you have filled in fields that will not apply?
I also had this issue and it took me around half an hour until I realized the user defined entrys were not supposed to look different from the prefilled ones and I was simply doing something wrong (I actually experimented with different separations like comma, semicolon, etc. until I realized what the correct method was).
The concept of turning the entrys into "tags" that can be deleted in one click instead of having to manually highlight and delete them is actually quite nice and now that I understand it I really like it. But it might need to be communicated more clearly to novice users, because (from my experience) it is not a common UX trope to have to "confirm" even your last entry in a space separated list with a trailing whitespace.

I'm not a UI/UX guy, but clearly also not the only one who took a while to get the way those entries work.

@Sarevok933219
Copy link

HI,

I'm testing the v2 image.

There is one wg interface configured in server mode.

  1. I set up an ip cidr in Edit interface -> Interface -> "IP Addresses" it not configured to the wg interface.
  2. So I set up a post-up hook to run ip command to assign IP address to the wg interface, but it seems the command not runned when I click to save.
  3. Int the Interface status for wg interface not shows the interface's IP address
  4. When I set up a peer with allowedIPs option the site saves it, but not set it up to the wg interface.

In the log I haven't seen any errors. This should be works, but I miss something? How can I debug it?
Add the inability to log in to the UI admin panel if Postgres is used as the backend

@h44z
Copy link
Owner

h44z commented Feb 1, 2024

Potential improvement: @h44z maybe there should be a help text or a warning when saving if you have filled in fields that will not apply? I also had this issue and it took me around half an hour until I realized the user defined entrys were not supposed to look different from the prefilled ones and I was simply doing something wrong (I actually experimented with different separations like comma, semicolon, etc. until I realized what the correct method was). The concept of turning the entrys into "tags" that can be deleted in one click instead of having to manually highlight and delete them is actually quite nice and now that I understand it I really like it. But it might need to be communicated more clearly to novice users, because (from my experience) it is not a common UX trope to have to "confirm" even your last entry in a space separated list with a trailing whitespace.

I'm not a UI/UX guy, but clearly also not the only one who took a while to get the way those entries work.

Nice suggestion. I need to think about how to implement this - I am not a UI/UX guy either ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants