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

Add a macro to use the MAC address as the gateway name. #1013

Merged
merged 1 commit into from
Jul 10, 2021

Conversation

h2zero
Copy link
Collaborator

@h2zero h2zero commented Jul 10, 2021

Description:

Adds the macro USE_MAC_AS_GATEWAY_NAME, when defined the gateway name will default to:
OMG_XXXXXXXXXXXX, where the X's are the mac address characters.
This can be changed in WiFiManager.

Checklist:

  • The pull request is done against the latest development branch
  • Only one feature/fix was added per PR and the code change compiles without warnings
  • I accept the DCO.

Adds the macro USE_MAC_AS_GATEWAY_NAME, when defined the gateway name will default to
OMG_XXXXXXXXXXXX, where the X's are the mac address characters.
This can still be changed in WiFiManager.
@1technophile 1technophile added this to the v0.9.7 milestone Jul 10, 2021
@1technophile 1technophile merged commit e92e468 into 1technophile:development Jul 10, 2021
@NorthernMan54
Copy link
Collaborator

@h2zero Tks very much for this, I had been doing it manually for a while now.

@NorthernMan54
Copy link
Collaborator

@h2zero I just updated my devices with the latest beta, and the OTA device name is not the expected Mac address but 'OpenMQTTGateway', shouldn't this use the Mac address as the OTA Device name as well ?

@h2zero
Copy link
Collaborator Author

h2zero commented Jul 15, 2021

@NorthernMan54 Thanks for pointing this out, it had not occurred to me to change this as well. I will submit a PR to rectify.

@h2zero h2zero deleted the mac-name branch July 17, 2021 14:26
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.

3 participants