agdsn / sipa

The Supreme Information Providing Application
MIT License
13 stars 10 forks source link

Add support for MPSK #498

Open mabezi opened 1 month ago

mabezi commented 1 month ago

In order for MPSK to be used by our members, a suitable frontend with connection to the Pycroft API (not yet available) must be implemented.

The user should be able to see a list of MAC addresses along with self-assigned device names. For better usability, it should not only be possible for the user to add or delete addresses but also to edit them when needed (e.g. to fix typos)

The following potential errors must be displayed in the frontend with appropriate messages:

The implementation can partially follow the existing process for changing MAC addresses.

It would be very useful to have a clearly visible notice indicating that the MAC address is not for the physical port but for WiFi (and vice versa).

It might also be worth considering placing the display of the WiFi password near the list, creating a major section called "WiFi." However, this is optional.

mabezi commented 1 month ago

Pycroft Issue: https://github.com/agdsn/pycroft/issues/744

lukasjuhrich commented 1 month ago

Will these devices also work via the agdsn ssid? I am thinking about how we communicate the difference between enterprise wifi and mpsk-wifi most effectively to the members.

I think some "branding" would really help here, also with respect to clarifying the distinction between "the agdsn wifi" vs. "your router's wifi". I'm thinking along these lines, where we would replace the W with a wifi icon:

image

I'm not really fixated on the color, this is a rough sketch. for the source see this tailwind prototype.

ibot3 commented 1 month ago

Will these devices also work via the agdsn ssid? I am thinking about how we communicate the difference between enterprise wifi and mpsk-wifi most effectively to the members.

There will be a separate SSID like agdsn-mpsk or similar. The agdsn SSID is for enterprise wifi with username/password only.

mabezi commented 1 month ago

Currently agdsn-mpsk is planned and configured as SSID.

lukasjuhrich commented 1 month ago

Okay, so: updated suggestion: https://play.tailwindcss.com/Or1PTD16dY image