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

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.

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