Skip to content

influxdb: add support for path parameter#13265

Merged
frenck merged 2 commits into
home-assistant:nextfrom
pdcemulator:next
May 3, 2020
Merged

influxdb: add support for path parameter#13265
frenck merged 2 commits into
home-assistant:nextfrom
pdcemulator:next

Conversation

@pdcemulator
Copy link
Copy Markdown
Contributor

Proposed change

My InfluxDB is running behind a reverse proxy, like https://myhost.mydomain.tld/influx.
To be able to access this db from HA, we have to add the new path parameter. The path parameter is already supported by the influxdb-python lib, which is used by this integration.

This is my first PR, so I hope I did everything right.

Type of change

  • Spelling, grammar or other readability improvements (current branch).
  • Adjusted missing or incorrect information in the current documentation (current branch).
  • Added documentation for a new integration I'm adding to Home Assistant (next branch).
  • Added documentation for a new feature I'm adding to Home Assistant (next branch).
  • Removed stale or deprecated documentation.

Additional information

Checklist

  • This PR uses the correct branch, based on one of the following:
    • I made a change to the existing documentation and used the current branch.
    • I made a change that is related to an upcoming version of Home Assistant and used the next branch.
  • The documentation follows the Home Assistant documentation standards.

@probot-home-assistant probot-home-assistant Bot added has-parent This PR has a parent PR in another repo next This PR goes into the next branch labels May 2, 2020
@pdcemulator pdcemulator changed the title added support for influxdb path parameter influxdb: add support for path parameter May 2, 2020
@frenck frenck added the parent-merged The parent PR has been merged already label May 3, 2020
Copy link
Copy Markdown
Member

@frenck frenck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @pdcemulator! 👍

@frenck frenck merged commit 65094e5 into home-assistant:next May 3, 2020
@probot-home-assistant probot-home-assistant Bot removed the parent-merged The parent PR has been merged already label May 3, 2020
shenxn pushed a commit to shenxn/home-assistant.io that referenced this pull request May 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

has-parent This PR has a parent PR in another repo next This PR goes into the next branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants