RileyXX / IMDB-Trakt-Syncer

A python script that syncs user watchlist, ratings and reviews for Movies, TV Shows and Episodes both ways between Trakt and IMDB.
MIT License
90 stars 2 forks source link

Unable to restart initial setup if incorrect input given #87

Closed MrCredible closed 6 months ago

MrCredible commented 6 months ago

I created this before I read the troubleshooting section of the github page.

I would suggest mentioning "deletion of credentials file if information is input incorrectly" when the syncer is first kicked off before asking for the initial trakt client ID.

But delete this issue, if its not needed.

Is there already an issue for your problem?

Bug Description

When installing via pip and running via cmd, if you input an incorrect value for one of the user inputs, you are unable to correct your previous input.

For example, if you provide the incorrect Trakt ID and push enter and realize after the fact that it was wrong, then you would most likely restart IMDBTraktSyncer. When you restart the syncer, it takes you back to where you left off.

Then if you decide to just give garbage inputs to quickly run through all the required user inputs, it will throw an exception once you reach the request for the URL authorization code. Once you restart, after the exception, it requests the same URL authorization code.

Uninstalling and reinstall does not allow you to start over.

Environment

windows 11 pip install ran via cmd

Screenshots

No response

RileyXX commented 6 months ago

Thanks very much for your feedback! I will be addressing this in an upcoming patch.

This problem came up for me a few times as well when I needed to check some things. Due to python limitations uninstalling the project won't remove any files that were created/modified by the script after installation which would be the credentials.txt and error log.txt files. When uninstalling a python project, python will actually prompt the user with the path(s) to any files that will not be deleted. This is one way you can quickly get the folder path.

In a future patch the script will print out the folder location where the project is installed every time the script runs. This should make it easier to quickly access the credentials.txt and error log.txt files in case of invalid credentials. Additionally I'll add better documentation so the user can easily find what do to in this situation.

RileyXX commented 6 months ago

For anyone else who comes across this issue, as pointed out by the original poster the solution for invalid credentials can be found in the troubleshooting section on the main github page:

If any of your details change (passwords, logins, API keys, etc.), simply open credentials.txt, modify your details, save it and then run the script again. Alternatively, delete credentials.txt to reset the script and it will prompt you to enter your new details on the next run.