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

Web UI issues / questions #732

Closed
gimi87 opened this issue Mar 26, 2018 · 4 comments
Closed

Web UI issues / questions #732

gimi87 opened this issue Mar 26, 2018 · 4 comments

Comments

@gimi87
Copy link

gimi87 commented Mar 26, 2018

I am building the firmware with the "custom.h" and "core_version.h" files where the devices have a prepared configuration by preprocessor instructions. Next i am uploading via ota and doing a factory reset, which should run the pre-build configuration. And then i am noticing:

  1. After overriding "MQTT_USER" preprocessor define, the MQTT is working, but in the web UI I see "Leave blank if no user".
  2. SONOFF_BASIC shows in the web UI's "HTTP port" field value: admin (it should show 80). I can't change this value.
  3. H801 shows in the web UI's "Hostname" field value: admin (it should show ESPURNA_xxxxxx). I can't change this value.

Additional question: Is there any chance to change the hostname via "#define" preprocessor instruction?
PS. I am using latest sources.

@mcspr
Copy link
Collaborator

mcspr commented Mar 27, 2018

First one yeah, thats a mqtt module bug only sending values to web from runtime settings and not built-in ones.

I cannot see 2 or 3 with factory reset device:

Can you show relevant parts of your custom headers and output of keys command? Can be run from Debug tab. Should be 1 key at most - hostname.

@gimi87
Copy link
Author

gimi87 commented Mar 27, 2018

Thanks, that gave me a clue. It turned out that the Chrome was autofilling the fields with "admin" :). Right now only the first point is valid.
And i am still waiting for chance to change hostname by "#define".

@xoseperez
Copy link
Owner

Thank you @mcspr ! Merged and ready to release.

@xoseperez
Copy link
Owner

Staged for release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants