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

Fix grdf without captcha, add load into HA LTS #23

Closed
wants to merge 1 commit into from

Conversation

vingerha
Copy link
Contributor

@vingerha vingerha commented Apr 20, 2024

I used the other PR "Release 1" to add grdf components and load into long-term-statistics
With the recent change of login at GRDF, I reshuffled the code to make it work without the captcha

Issues I had with loading config.json elements. In some cases self.configuration([PARAM]) worked and in others it did not so used configuration_json.get([PARAM]). As this is not just with 'my' code I assume the original py has introduced that....is confusing a lot.
For now use_captha is hard-set to False but I would prefer to have this via the json, same as for allowing to print screenshot.

# Wait for Password #####
self.mylog("Waiting for Password", end="")

ep = EC.presence_of_element_located((By.ID, "input54"))
Copy link
Contributor

Choose a reason for hiding this comment

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

For me the IDs were changing on each refresh. It may be more stable to rely on input types or input name
I used this for password

            ep = EC.presence_of_element_located(
                (By.XPATH, r"//input[@name='credentials.passcode']")
            )

and this for email

           EC.presence_of_element_located(
                (By.XPATH, r"//input[@name='identifier']")
            )

@vingerha
Copy link
Contributor Author

I have abandoned this and rebuillt gazpar2mqtt as it was too cumbersome to also have Veolia for which I have 0 interest.

@vingerha vingerha closed this Aug 30, 2024
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.

2 participants