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

Punch fix #63

Merged
merged 3 commits into from
Jan 19, 2024
Merged

Punch fix #63

merged 3 commits into from
Jan 19, 2024

Conversation

vitox013
Copy link

@vitox013 vitox013 commented Jan 18, 2024

Fixed a bug that prevented punch cards from being completed (it would only enter the screen and immediately exit without clicking on the challenges). The issue was with the line self.webdriver.find_element(By.CLASS_NAME, "offer-cta").click() since it was not clickable due to its height and width being 0, causing an error when attempting to click: "element not interactable: element has zero size." Sometimes the execution would continue, and other times it would break. Therefore, I modified the child element of the tag to be clicked instead.

@vitox013
Copy link
Author

It's my first pull request, and I'm not sure why it shows that I changed 3 files. I only made modifications to the file src/punchCards.py.

@Titaniumtown
Copy link

@vitox013 the branch you're trying to merge is outdated. As in it isn't up-to-date with the main repository. Inside your directory you can run
git pull https://github.com/klept0/MS-Rewards-Farmer --rebase and it will rebase your commit(s) on the latest commits on this repo.

nicoacu and others added 3 commits January 18, 2024 01:16
Fixed a bug that was preventing the execution of punch cards, as the <a /> tag was identified, but an error occurred when attempting to click: "element not interactable: element has zero size." Therefore, I modified the child element of the <a /> tag to be clicked instead.
@klept0 klept0 merged commit a759efb into klept0:master Jan 19, 2024
4 checks passed
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.

4 participants