-
Notifications
You must be signed in to change notification settings - Fork 13
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
Change menu item text #19
base: develop
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Tom,
I agree on the need to have clearer menu titles. This issue dates back a long way and there is little consistency (particularly in the code). My comments:
Setup -> Server Defaults? SetUp implies initialising?
Models and Keysets -> Hardware Defaults
Extensions and Devices -> Extensions and Phones
@dkgroot any comments
maybe if someone has the commercial EPM we could base the structure from that
…-- Tom
________________________________
From: steve-lad ***@***.***>
Sent: Tuesday, June 15, 2021 5:16:42 PM
To: chan-sccp/sccp_manager ***@***.***>
Cc: 19226484 - Tom Quinn ***@***.***>; Author ***@***.***>
Subject: Re: [chan-sccp/sccp_manager] Change menu item text (#19)
@steve-lad commented on this pull request.
Thanks Tom,
I agree on the need to have clearer menu titles. This issue dates back a long way and there is little consistency (particularly in the code). My comments:
Setup -> Server Defaults? SetUp implies initialising?
Models and Keysets -> Hardware Defaults
Extensions and Devices -> Extension and Phones
@dkgroot<https://github.com/dkgroot> any comments
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<#19 (review)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ARHTUUYRFA7QQFARNEQVWDLTS54GVANCNFSM46W5GKIQ>.
|
Hi Steve and Tom Yes i agree that clear names could help. Setup -> Server Defaults? SetUp implies initialising? Models and Keysets -> Hardware Defaults Extensions and Devices -> Extensions and Phones
If chan-sccp had called them 'Phone' and 'Extension' then maybe/sure, but that ship has sailed. If you really do want to mix these terms then the page what comes up ought to have a little explanation bubble at the top to explain that: So my tip would be go with "Lines and Devices" I know it sucks, but it lines up nicely with what's in the CLI, and sooner or later even freepbx users will have a look at the CLI (I think) Naming Stuff Is HARD @tomquinn04 Thank you for your contribution and raising the point. Bit of open discussion via an PR is always welcome. |
@dkgroot |
@tomquinn04 Line's and Extensions are not the same thing though. They are related, but that relation can be tight or quite loose, depending on their use / history. For example some voip environments have phones/lines tied to floor and office numbers they are in, but the extensions in that firm are historically tied to the user, in which case they would not resemble each other at all and would be mapped/linked to one another. And to tie it all off the external DID (Direct Inward Dialing) numbers in that firm are not directlly linked to the internal extension so they would be mapped as well. I think it's important to keep these names separately. I guess i will let steve decide what he want's to do. |
No description provided.