search

freifunkh / site

Freifunk Hannover specific Gluon site configuration for Firmware building.
https://hannover.freifunk.net
5 stars 5 forks source link

Added Umland and Region Prefix to Domain Names #61

Closed Dark4MD closed 3 years ago

Dark4MD commented 3 years ago

Wie vor einiger Zeit (Monate? Habs etwas aus dem Auge verloren) besprochen hier ein Pull Request um die Benennung der Domains in der Firmware etwas anzupassen. Eingeteilt nun wie auch die "Hannover" Stadtteile mit Umland und Region Präfix.

Sollten keine beanstandungen sein würde ich mich mit der Änderung endlich mal an eine vH22 machen.

lemoer commented 3 years ago

Ich sehe da einige Probleme:

  1. Die Präfixe wären inkonsistent. Konsequent wären imho z.B.:
    • Möglichkeit a) "Stadt Hannover: ...", "Region Hannover: ...", "Umland: ..."
    • Möglichkeit b) "Stadt: ", "Region: ", "Umland: ".
  2. Die Liste wird unübersichtlicher. Wenn man nicht weiß, dass es die Kategorien "Region" und "Umland" gibt, dann sucht man ggf. an der falschen Stelle, oder so?

Kannst du nochmal erklären, wie du auf die Idee für diesen PR gekommen bist? Wo wären die Vorteile von diesem Prefixing?

Dark4MD commented 3 years ago

Die Idee ist im Grunde hier von @CodeFetch entstanden und sollte die bennenung anpassen an die bereits bestehenden "Hannover: ..." Einträge: https://github.com/freifunkh/site/issues/57#issuecomment-784706052

AiyionPrime commented 3 years ago

mit Umland und Region Präfix.

Ich glaube nicht, dass das zur Übersichtlichkeit beiträgt.

Die Idee ist im Grunde hier von @CodeFetch entstanden

Überzeugt mich aus den von Leo genannten Gründen trotzdem nicht. @Codefetch ist dir das wichtig? Dann sollten wir das ausdiskutieren...

CodeFetch commented 3 years ago

@AiyionPrime Die Liste wird ja alphabetisch angezeigt. Das wäre schon sinnvoll. Mir ging es darum, dass Umland mittlerweile über 300 Knoten hat und man das gut mit der Region Hannover abfangen könnte.

AiyionPrime commented 3 years ago

Dass die Liste alphabetisch angezeigt wird, ist richtig, aber ja nicht der Punkt. Jemand der einen Router in Alfeld aufstellt, sucht Alfeld unter A, nicht unter U für Umland oder R für Region oder dem nächsten Buchstaben den wir uns ausdenken.

Schlimmer ist allerdings, dass so jemand nichtmal notwendigerweise die Kategorien Region oder Umland sieht.

tobby88 commented 3 years ago

Sinnvoll finde ich die Eingruppierung, ja. Aber als Sortierung vermutlich eher ungeeignet. Wenn ich in Barsinghausen wohne, suche ich vermutlich zuerst unter B wie Barsinghausen. Finde nichts. Scrolle die Liste bis nach ganz unten, sehe Umland, finde Barsinghausen nicht, suche weiter, finde Region, finde Region: Barsinghausen. Hypothetisches Beispiel, das zeigen soll, wie wenig intuitiv eine Gruppierung sein kann. Daher wäre ich dafür, bei der alten Sortierung ohne Gruppierung zu bleiben. Einzige Variante, die das verbessern würde, wäre es, auch alle bisher ungruppierten Regionen/Stadtteile auch zu gruppieren. Aber auch dann weiß ich nicht auswendig, ob Barsinghausen jetzt zur Region oder zum Umland gehört und suche immer noch in mindestens zwei Kategorien. Daher: dagegen.

CodeFetch commented 3 years ago

Scrolle die Liste bis nach ganz unten, sehe Umland, finde Barsinghausen nicht, suche weiter, finde Region

@tobby88 R kommt vor U. Von daher schon sinvoll, da es quasi von Hannover immer weiter entfernt wird. Aber ja... Man könnte mal nen Gluon-Patch bauen, um das systematisch eingeben zu können.

1977er commented 3 years ago

Ich find die Präfixe auch eher unintuitiv und würde sie weglassen (also den status quo beibehalten).

AiyionPrime commented 3 years ago

@Dark4MD Damit bleibt nur noch https://github.com/freifunkh/site/pull/61/files#diff-75d0c508a8e6befc8c791cc460b1750b3d04fe721ff722c64e012d3ac063e324R15 richtig?

Dark4MD commented 3 years ago

Habs angepasst, hab erstmal Leetfeld als neues Default gesetzt. Müsste eventuell nochmal diskutiert werden ob wir das so lassen oder eine andere (neue?) default Domain setzen.