Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix for missing emoncms_path #85

Merged
merged 1 commit into from
Dec 14, 2020

Conversation

fillycheezstake
Copy link
Contributor

This fixes the issue where the emoncms path is ignored in the webui and not used to generate emoncms requests.

This fixes the issue where the emoncms path is ignored in the webui and not used to generate emoncms requests.
@glynhudson glynhudson merged commit 84e4929 into openenergymonitor:master Dec 14, 2020
@glynhudson
Copy link
Member

Thanks 👍

@glynhudson
Copy link
Member

I'm just testing this PR, and it seems to result in the module unable to boot and entering a bootloop. Have you tested this PR?

@glynhudson
Copy link
Member

I've fixed it, it required String url = emoncms_path.c_str();

84f7800

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants