FastEddyCurrent / zap2xml

zap2xml in Python 2.7 for use on the RaspberryPI
10 stars 6 forks source link

zap2it had some web changes that break login... #10

Closed edit4ever closed 8 years ago

edit4ever commented 8 years ago

Looks like they are now using a popup for login...any thoughts on how to fix the script?

edit4ever commented 8 years ago

The fix is to update the login match string:

Change line 148 to:

loggedinStr = '.Logout of your Screener account.'

My repo has a different folder structure, so I can't submit a pr, but I wanted to send you the fix.

Schmell commented 8 years ago

im not sure i follow. The script runs as expected. No pop up here???

On Thu, Oct 20, 2016 at 11:13 AM, edit4ever notifications@github.com wrote:

Looks like they are now using a popup for login...any thoughts on how to fix the script?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/FastEddyCurrent/zap2xml/issues/10, or mute the thread https://github.com/notifications/unsubscribe-auth/AD7IwfPd66WuGjaghSGpQaqI3nMhl4qYks5q14UugaJpZM4KcP8T .

Thanks Sheldon

edit4ever commented 8 years ago

Which script are you running? I updated my code with the new website name reference - so it runs correctly and it looks like Fast Eddy updated his as well. Basically look at line 148 of the zap2xml.py file.

If it reads: loggedinStr = '.Logout of your Screener account.' you're running the current version and won't have any issues.

If it reads: loggedinStr = '.Logout of your Zap2it account.' it won't work anymore.

My guess is you have the updated script. :-)

Schmell commented 8 years ago

Version 0.4.6 Nice Fix. Its working very well. You should submit it to a repo so updates can be automated