Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Script Updated and enhaced. #21

Open
wants to merge 18 commits into
base: master
Choose a base branch
from
Open

Conversation

Gabo-Tech
Copy link

No description provided.

@reahaasVR
Copy link

@Gabo-Tech I looked at your code and it seems like really good additions, that makes the project better than the original code.

@coding-ai Why is it not approved? Is the repository is still alive? If not, maybe migrate it to your repo and maintain it alive?

@reahaasVR
Copy link

reahaasVR commented Nov 2, 2024

1 note about the driver:
In your README file, you mentioned that it is only necessary to download a different driver and update the path in the config file.
But that is not enough. You also need to use different Driver objects, for example for chromedriver:

from selenium.webdriver.chrome.service import Service as `ChromeService` instead of the `FirefoxService`
...
class EasyApplyLinkedin:
    ...
    def __init__(self, data):
        ...
        chrome_service = ChromeService(executable_path=data["driver_path"])
        self.driver = webdriver.Chrome(service=chrome_service)
        ...

@Gabo-Tech
Copy link
Author

I think it has been abandoned. I forked it. Thanks for the addition. Feel free to create a pull request. to my fork or here however you prefer, but yeah here I believe no one will see it.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants