pawelhulek / pgnig-sensor

This sensor is gathering gas usage data from PGNIG ebok page.
MIT License
32 stars 8 forks source link

Zdublowane dane płatności dla dwóch liczników #17

Closed Tomar16pl closed 2 years ago

Tomar16pl commented 2 years ago

Na początku wielkie dzięki za stworzenie tej integracji! Świetna robota!

Posiadam w PGNiG zarejestrowane 2 liczniki gazu. Integracja poprawnie odczytała stany obu z nich, lecz niestety dane dotyczące nadchodzących płatności są zdublowane. W eBOK widzę, że na jednym koncie mam niedopłatę 0,14 zł, na drugim koncie stan płatności jest równy 0. W HA integracja na obu licznikach pokazuje tą samą wartość :-(

pawelhulek commented 2 years ago

@Tomar16pl Moglbys przetestowac na wersji 1.04-beta? Mozna ja wybrac po kliknieciu na beta program przy intergacji. Jesli wszytko bedzie OK, zmerguje do maina.

Tomar16pl commented 2 years ago

@pawelhulek, dzieki za info.  Zainstalowałem wersję 1.04-beta i niestety dla obu liczników encja z wartościami faktur jest niedostępna. Encje z wysokością zużycia wyświetlają poprawne dane.  Zastanawiam sie czy przypadkiem powodem dla którego encje z wysokościami faktur są niedostępne jest to, że na jednym z kont (tym z zielonym R)  jest zaległość 14 gr, przez co de facto dla 2 liczników w systemie są 3 rachunki. Zrzut z panelu PGNiG poniżej ;-).

Dzięki!

W dniu 08.05.2022 13:56:45, pawelhulek @.> pisze: @Tomar16pl [https://github.com/Tomar16pl] Moglbys przetestowac na wersji 1.04-beta? Mozna ja wybrac po kliknieciu na beta program przy intergacji. — Reply to this email directly, view it on GitHub [https://github.com/pawelhulek/pgnig-sensor/issues/17#issuecomment-1120404793], or unsubscribe [https://github.com/notifications/unsubscribe-auth/AQHF4LTLMRHECPUNJ47WJSDVI6TXZANCNFSM5UTCG4IQ]. You are receiving this because you were mentioned.Message ID: @.> [fcefad49-92bd-4a3b-851a-fc040a1e28b8]

Tomar16pl commented 2 years ago

screenshot

pawelhulek commented 2 years ago

Hej, Mialem drobny blad w kodzie opublikowałem wersje 1.0.5-beta

Tutaj bardziej zastanawia mnie jak zmatchowac dane licznika z danymi faktury.

Osobiscie mam tylko jeden licznik i ciezko mi przewidziec jak PGNIG te dane zamodelowało. Jak nie uda sie na wersji 1.0.5-beta to bede musial Cie prosic o analize po jakim polu moge zmatchowac fakture z licznikiem.

Przygotuje instrukcje.

Narazie prosiłbym o sprawdzenie wersji 1.0.5-beta

Tomar16pl commented 2 years ago

Cześć, Po aktualizacji do 1.0.5-beta pojawiła się wartość dla konta 'czerwonego'. Dla zielonego nadal wartość encji 'niedostępna'.

Podeślij proszę instrukcję to postaram się pomóc. Dzięki!

pawelhulek commented 2 years ago

Otworz chrome i stronke https://ebok.pgnig.pl/ kliknij prawym nastepnie opcja inspect (na samym dole rozwinietego okna) Nastpenie przejdz w zakładkę network (siec). Odswiez strone wejdz na zakladke faktury. Teraz interesuja nas 2 zapytania crm/get-ppg-list oraz crm/get-invoices-v2 Po znaznaczeniu ich i przejscu w preview powinienes zobaczyc co zwróciły. I teraz majac juz dane z tych endpointow bede Cie prosił o znalezienie pasujacych do siebie pól pomiedzy faktura a licznikiem. U mnie pasuje np pole idLocal z getPpgList oraz IdPP z getInvoices Natomiast nie wiem jak to wyglada kiedy ma sie wiecej licznikow. Podejzewam ze moze to byc np cotract number i agreement number.

@Tomar16pl Wolałbym zebys nie zamieszczał tutaj zwrotki z endpointow. Pamietaj ze tam sa twoje personalne dane.

Pozdrawiam,

Tomar16pl commented 2 years ago

Nie znalazłem u siebie zapytania crm/get-ppg-list, mam natomiast get-agreements (w dwóch instancjach) oraz get-all-agreements, które jako parametr Typeprzesyłają wartość "PPG". Nie znalazłem tam jednak pola idLocal. Znalazłem natomiast inne pole Iban, ktore występuje w obu tych zapytaniach. Jeśli chcesz, to mogę podrzucić zrzuty z zapytań, oczywiście po zasłonięciu danych prywatnych ;-).

pawelhulek commented 2 years ago

@Tomar16pl Dziwne ze nie widzisz ppg-list.

Zaloguj sie prosze w chrome do pging i podmien adres strony na https://ebok.pgnig.pl/crm/get-ppg-list?api-version=3.0

chudokOne commented 2 years ago

Witam, na wstepie dzieki za wklad pracy i fajne rozwiazanie! Chcialbym pomoc z tymi wielokrotnymi fakturami, osobiscie mam 3 podpiete na tym samym koncie. Potwierdzam wszystkie uwagi zwrocone przez przedmowce, tzn. nie znajduje encji get-ppg-list, nie jest tez mozliwe uzyskanie odpowiedzi poprzez wpisane w pasek adresu, bo tylko zwraca standardowy komunikat bledu w takim przypadku. Korzystajac z inspektora udalo mi sie wyszukac potrzebne dane w encji get-invoices-v2. Jej struktura jest uwidoczniona na zrzucie ekranu ktory mam nadzieje uda sie dolaczyc do tego wpisu. Przepraszam za slaba jakosc zrzutu.

W dniu 16 maj 2022, 06:16, o 06:16, użytkownik pawelhulek @.> napisał: @.

Dziwne ze nie widzisz ppg-list.

Zaloguj sie prosze w chrome do pging i podmien adres strony na https://ebok.pgnig.pl/crm/get-ppg-list?api-version=3.0

-- Reply to this email directly or view it on GitHub: https://github.com/pawelhulek/pgnig-sensor/issues/17#issuecomment-1127198953 You are receiving this because you are subscribed to this thread.

Message ID: @.***>

Tomar16pl commented 2 years ago

Po podmianie adresu wyświetliła mi się odpowiedź na zapytanie i pasują pola:

idLocal z getPpgList oraz IdPP z getInvoices ale również ContractNumber zgetPpgList oraz AgreementNumber z getInvoices

Wrzucam jeszcze listę wszystkich zapytań z 'get', które widzę w inspektorze:

zapytania_get

pawelhulek commented 2 years ago

@Tomar16pl

Wybacz poślizg w rozwiązywaniu, ostatnio nie mam zbytnio na to czasu ;)

Sprawdziłbyś prosze jeszcze ta fakture przy odpowiedzi getInvoices. Szczegolnie interesuje mnie jaka wartość ma pole "status" ;) Chociaz obawiam sie ze juz ja mogles opłacić i sie nie dowiem ;)

pawelhulek commented 2 years ago

@Tomar16pl Opublikowalem kolejna betę [2.0.0-beta](https://github.com/pawelhulek/pgnig-sensor/releases/tag/2.0.0-beta)

Tylko tutaj ostrzegam ze zmieniaja sie idki encji. Mam nadzieje ze nie masz automatyzacji opartych na tych encjach

Tomar16pl commented 2 years ago

@pawelhulek - cieszę się, że nie zarzuciłeś tematu :-)

Niestety obie faktury zostały już opłacone, więc wszędzie mam status "Posted".

Zrobiłem update do 2.0.1-beta i niestety wszystkie 4 encje mam niedostępne (w narzędziach deweloperskich tylko te 4 stare mi wyszukuje gdy szukam frazy PGNIG)

pawelhulek commented 2 years ago

@Tomar16pl to ciekawe, ze Ci nowe encje nie wskoczyly.

Masz moze jakies logi z HA?

(ustawienia system logi)

Dodatkowo powiedz mi czy masz integracje konfigurowana z UI czy z configuration.yaml?

Tomar16pl commented 2 years ago

W logach HA znalazłem jedynie to:

Logger: homeassistant.components.sensor
Source: custom_components/pgnig_gas_sensor/sensor.py:78
Integration: Sensor (documentation, issues)
First occurred: 31 maja 2022, 21:53:41 (1 occurrences)
Last logged: 31 maja 2022, 21:53:41

Error while setting up pgnig_gas_sensor platform for sensor
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 249, in _async_setup_platform
    await asyncio.shield(task)
  File "/config/custom_components/pgnig_gas_sensor/sensor.py", line 64, in async_setup_platform
    [PgnigSensor(hass, api, meter_id, x.id_local), PgnigInvoiceSensor(hass, api, meter_id, x.id_local)],
  File "/config/custom_components/pgnig_gas_sensor/sensor.py", line 78, in __init__
    self.entity_name = "PGNIG Gas Sensor " + meter_id + " " + id_local
TypeError: can only concatenate str (not "int") to str

Integrację konfigurowałem poprzez configuration.yaml.

pawelhulek commented 2 years ago

ajajaj. Wypuscilem zla wersje ;( Lokalnie smigało, bo nie wypushowalem zmian.

Release 2.0.2-beta powinna smigac.

Sprawdz prosze przy nastepnym odczycie z licznikow czy sie dobrze rozkladaja faktury

Tomar16pl commented 2 years ago

Teraz jest OK :-).

Liczniki odczytam dziś/jutro więc feedback będzie do kilku dni.

Dzięki raz jeszcze za dobrą robotę :)

Tomar16pl commented 2 years ago

@pawelhulek, Dziś pojawiły się nowe faktury i tym razem wyświetlają się poprawnie zarówno stany liczników jak i faktury :-). Dzięki raz jeszcze i pozdrawiam!