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

Use InfluxDb 2 write api #12

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

aeytom
Copy link

@aeytom aeytom commented May 1, 2022

  • rename module to "InfluxDb2Stats"
  • config uses: organisation, bucket, token
  • write to influx db using /api/v2/write endpoint

- rename module to "InfluxDb2Stats"
- config uses: organisation, bucket, token
- write to influx db using /api/v2/write endpoint
@mrohn
Copy link

mrohn commented May 6, 2022

I did this changes also on my side. I would like to see the Z-Way app updated in the App Store to make it available also for other users. But to do this without breaking existing installation there should be a new App added to the App Store or the implementation should be able to be configured to the old and the new one.
What are you thinking about this or are you not interested in an update on the App Store?

@TheTwistedTransistor
Copy link

TheTwistedTransistor commented Dec 31, 2022

I did this changes also on my side. I would like to see the Z-Way app updated in the App Store to make it available also for other users. But to do this without breaking existing installation there should be a new App added to the App Store or the implementation should be able to be configured to the old and the new one. What are you thinking about this or are you not interested in an update on the App Store?

That would be very appreciated!

also not really sure how to activate the fork once downloaded with git. When only replacing the folder new fields like bucket etc are not replaced with database etc in the frontend.

@TheTwistedTransistor
Copy link

got it running now. Nice job! But I think a merge does not makes sense as the former version is also for the old influxDB version. Can we bring it somehow to the appstore as an additional influxDB2 App maybe?

"Installation" worked for me to drop the source code into /opt/z-way-server/automation/userModules/InfluxDb2Stats. I had to restart the z-way-server and the app appears in the local apps. It seems to have a problem with the language file, I commented out the line "this.langfile = undefined;" helped to get the message away. Now I only get an UI Error accessing the webfrontend. But I can click it away...

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