WebDriverException not defined #139

Merged
thomascrha merged 1 commits from patch-1 into master 4 years ago
thomascrha commented 4 years ago (Migrated from github.com)
Owner

I didn't have chromedriver installed initially - it caused a secondary exception.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/tcrha/.local/bin/deletefb", line 8, in <module>
    sys.exit(run_delete())
  File "/home/tcrha/.local/lib/python3.8/site-packages/deletefb/deletefb.py", line 112, in run_delete
    driver = login(
  File "/home/tcrha/.local/lib/python3.8/site-packages/deletefb/tools/login.py", line 44, in login
    driver = setup_selenium(chrome_options, chrome_binary_path)
  File "/home/tcrha/.local/lib/python3.8/site-packages/deletefb/tools/chrome_driver.py", line 66, in setup_selenium
    except WebDriverException:
NameError: name 'WebDriverException' is not defined
I didn't have chromedriver installed initially - it caused a secondary exception. ``` During handling of the above exception, another exception occurred: Traceback (most recent call last): File "/home/tcrha/.local/bin/deletefb", line 8, in <module> sys.exit(run_delete()) File "/home/tcrha/.local/lib/python3.8/site-packages/deletefb/deletefb.py", line 112, in run_delete driver = login( File "/home/tcrha/.local/lib/python3.8/site-packages/deletefb/tools/login.py", line 44, in login driver = setup_selenium(chrome_options, chrome_binary_path) File "/home/tcrha/.local/lib/python3.8/site-packages/deletefb/tools/chrome_driver.py", line 66, in setup_selenium except WebDriverException: NameError: name 'WebDriverException' is not defined ```
weskerfoot commented 4 years ago (Migrated from github.com)
Owner

looks good to me, thanks!

looks good to me, thanks!
weskerfoot commented 4 years ago (Migrated from github.com)
Owner
released https://pypi.org/project/delete-facebook-posts/1.1.17/ to fix this
The pull request has been merged as c1af411453.
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.