search

paprins / omnik-data-logger

GNU General Public License v3.0
1 stars 3 forks source link

Invalid URL #1

Closed jbouwh closed 4 years ago

jbouwh commented 4 years ago

I found out that there is a syntax error in client.py: File: https://github.com/paprins/omnik-data-logger/blob/master/omnik/client.py With pip3 install this file is located here at my distribution: /usr/local/lib/python3.7/dist-packages/omnik/client.py

Line: 55

is:
url = '{self.base_url}/plant/list' should be: url = f'{self.base_url}/plant/list'

After this patch the client works and submists the data to pvoutput.org.

paprins commented 4 years ago

Hi, must have been a glitch ... my bad, you're right. Will have a look at your PR and merge it.

paprins commented 4 years ago

Fixed issue ... keeping PR open for (future) Home Assistant compatibility.