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

[influxdb] allow defining measurement name in item metadata and code improvements #71

Merged
merged 6 commits into from
Apr 18, 2021

Conversation

J-N-K
Copy link
Member

@J-N-K J-N-K commented Apr 17, 2021

@J-N-K J-N-K added the enhancement New feature or request label Apr 17, 2021
@J-N-K J-N-K added this to the 3.1.1 milestone Apr 17, 2021
- Add option for using metadata value as measurement name
- Code improvements

Also-By: Johannes Ott <info@johannes-ott.net>
Signed-Off-By: Jan N. Klug <jan.n.klug@rub.de>
Signed-off-by: Jan N. Klug <jan.n.klug@rub.de>
@J-N-K J-N-K changed the title [influxdb] allow defining measurement name in item metadata [influxdb] allow defining measurement name in item metadata and code improvements Apr 18, 2021
J-N-K added 4 commits April 18, 2021 11:34
Signed-off-by: Jan N. Klug <jan.n.klug@rub.de>
Signed-off-by: Jan N. Klug <jan.n.klug@rub.de>
Signed-off-by: Jan N. Klug <jan.n.klug@rub.de>
Signed-off-by: Jan N. Klug <jan.n.klug@rub.de>
@J-N-K J-N-K merged commit 1d4815c into smarthomej:main Apr 18, 2021
@J-N-K J-N-K deleted the influxdb branch April 18, 2021 16:50
J-N-K added a commit that referenced this pull request Apr 18, 2021
…improvements (#71)

* metadata improvements

- Add option for using metadata value as measurement name
- Code improvements

Also-By: Johannes Ott <info@johannes-ott.net>
Signed-Off-By: Jan N. Klug <jan.n.klug@rub.de>

* refactoring

Signed-off-by: Jan N. Klug <jan.n.klug@rub.de>

* use checked exception

Signed-off-by: Jan N. Klug <jan.n.klug@rub.de>

* fixes

Signed-off-by: Jan N. Klug <jan.n.klug@rub.de>

* fix

Signed-off-by: Jan N. Klug <jan.n.klug@rub.de>

* Upgrade dependencies to fix issue with closing influx connections

Signed-off-by: Jan N. Klug <jan.n.klug@rub.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant