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

Added Umland and Region Prefix to Domain Names #61

Merged
merged 1 commit into from
Jul 22, 2021

Conversation

Dark4MD
Copy link
Member

@Dark4MD Dark4MD commented Jun 3, 2021

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.

@Dark4MD Dark4MD requested review from lemoer, 1977er and AiyionPrime June 3, 2021 21:43
@lemoer
Copy link
Contributor

lemoer commented Jun 3, 2021

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
Copy link
Member Author

Dark4MD commented Jun 4, 2021

Die Idee ist im Grunde hier von @CodeFetch entstanden und sollte die bennenung anpassen an die bereits bestehenden "Hannover: ..." Einträge:
#57 (comment)

@AiyionPrime
Copy link
Member

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
Copy link

@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
Copy link
Member

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
Copy link
Contributor

tobby88 commented Jun 14, 2021

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
Copy link

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
Copy link
Member

1977er commented Jun 18, 2021

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

@AiyionPrime
Copy link
Member

@Dark4MD
Copy link
Member Author

Dark4MD commented Jul 12, 2021

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.

site.conf Outdated Show resolved Hide resolved
@Dark4MD Dark4MD merged commit 2258cae into freifunkh:master Jul 22, 2021
Dark4MD added a commit to Dark4MD/site that referenced this pull request Sep 6, 2022
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

Successfully merging this pull request may close these issues.

6 participants