Sonoff-Tasmota-Modified icon indicating copy to clipboard operation
Sonoff-Tasmota-Modified copied to clipboard

Sonoff-SC missing friendly name on Information page

Open mike2nl opened this issue 5 years ago • 2 comments

Used tools:

  • Notepad++ v7.5.7

Used Hardware:

  • SONOFF-SC

Have you look for this feature in other issues and in the wiki? -> Yes.

Is your feature request related to a problem? Please describe. A clear and concise description of what the problem is. -> When you have the SOnOff-SV in use, a envoirement sensor, you will miss the friendly name on the information web page which can be a little irritating because when you have set the hostname different you miss it.

Describe the solution you'd like A clear and concise description of what you want to happen. -> I have changed a very little part in the xdrv_02_webserver.ino file to add this information and it works perfect.

Original Code: if (SONOFF_IFAN02 == Settings.module) { maxfn = 1; }

Changed Code: if ((SONOFF_IFAN02 == Settings.module) || (SONOFF_SC == Settings.module)) { maxfn = 1; }

Additional context Add any other context or screenshots about the feature request here. ... i'm willing to help with much more ...

mike2nl avatar Aug 01 '18 08:08 mike2nl

this is also wanted, cuz the friendyname is ONLY for alexa devices.

if their is no relais to switch, their will be no friendlyname to setup.

and in the line before the maxfn is defined by teh count of devices/relaises. in you change you overwrite and hardcoded the maxfriendlyname to 1, which means you will never be able to set more friendlynames if you (dont forget other users) add another relais to the modul.

reloxx13 avatar Aug 15 '18 22:08 reloxx13

I see but that is the original tasmota code not mine! I have added that here only: || (SONOFF_SC == Settings.module))

Because the original code: if (SONOFF_IFAN02 == Settings.module) { maxfn = 1; } It is a max of one. So where is the issues here? Possible it's to early today to see the issue.

And possible you can help me out to understand what you mean.

So in my eyes i see no issues with that because that changing does no harm. I have now 223 sonoff devices and i see no issues at all

mike2nl avatar Aug 16 '18 09:08 mike2nl