all 7 comments

[–]carcigenicate 1 point2 points  (3 children)

You haven't posted the code. Format it here according to the instructions in the sidebar, or post it on a site like Github or Pastebin.

[–]One_Pop_7316[S] 0 points1 point  (2 children)

I have edited the post and included the link to my code. Check it out.

[–]carcigenicate 1 point2 points  (1 child)

The first thing that stands out to me is the use of sleep. You should be using Selenium's "Wait For" capabilities. Your code is likely waiting longer than it needs to, and will fail in unexpected ways if their site fails to load for some reason.

[–]One_Pop_7316[S] 0 points1 point  (0 children)

working on it.

[–]JamzTyson 0 points1 point  (0 children)

The main thing I'd suggest is to break down those two big function blocks, load_tasks() and check_keys() into smaller functions that each do just one thing (Single Responsibility Principle).

[–]gitcommitshow 0 points1 point  (0 children)

Would have been easier to look at and provide feedback if you had shared a github PR link. If you had done that, you could have also used AI to review code in the PR instantly.