Conversation
Updated for GUI information
|
@mnai01 You've got some conflicts after the latest merge to master. |
|
Not a problem, I'll fix it thanks. |
Also, the last merged PR added some new script options and whatnot... so keep that in mind. |
Ye I see in addition to CVV multiple options were added so I'll have to add that to the GUI. |
GUI.py
Outdated
| ), url=self.url_entry.get(), shoe_size=self.shoe_size.get(), cvv=self.cvv_entry.get(), purchase=purchase, release_time=release_time, page_load_timeout=page_load_timeout) | ||
|
|
||
| # Determines if bot features where disabled. | ||
| # if(self.login_time_int.get() != 1): |
There was a problem hiding this comment.
Any reason why all this is commented out?
There was a problem hiding this comment.
Yes, its old code that was used to edit the console command options executed by os.command() but that method is no longer being used so I have been using that code as a reference to create the new method which just calls the run function directly with the set options. Totally forgot that part wasn't fully converted, still working on setting all options in the run function. The comments will be removed in the next commit
main.py
Outdated
| parser.add_argument("--shoe-type", default="M", | ||
| choices=("M", "F", "W", "Y", "C")) | ||
| parser.add_argument("--webdriver-path", required=False, default=None) | ||
| parser.add_argument("--cvv", required=False) |
There was a problem hiding this comment.
I think you screwed something up when you resolved conflicts. You're missing
--shipping-address and --shipping-option. Also --cvv has changed a bit in this PR. Does this file even need to be changed at all if you're just adding a separate GUI?
There was a problem hiding this comment.
The main.py wont need to be modified at all. I already updated my main.py to the latest PR. Right now i'm in the process of adding all the GUI code. The reason those are missing is because when I was looking at @alfredog1976 PR I only focused on adding the CVV since that's what I needed for my drops and at the time it wasn't pushed to the master yet.
| * If specified, will use the specified driver instead of the defaults | ||
| * NOTE: The driver should match the browser specified in the `--driver-type` option (defaults to Firefox) | ||
|
|
||
| <b>--cvv</b> |
There was a problem hiding this comment.
This is already described in the README.
baojieAngel
left a comment
baojieAngel
left a comment
There was a problem hiding this comment.
I think I should take a serious look, is this function already implemented
It works now since it has all of the required options, but I'm still in the process of adding every other option. Have been busy with work and vacation so it's been taking awhile. Should be on it soon. |
3 participants
In-depth GUI with multiple options and cvv from @alfredog1976. I will be adding more options throughout time to make it even easier for people to run.