-
Notifications
You must be signed in to change notification settings - Fork 184
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Not running under Windows10 #1
Comments
Sounds a bit like the login is being rejected, you don't have the "<" or ">" around the token right? And the token came from https://discordapp.com/developers where you register an "app" under "my apps" on the left and find the token correct? |
Also just pushed an update for logging of errors, so it should create a file in the base directory called "rhythm-bot-log.log" to sort of assist with future errors. |
The ">" and "<" characters are not present in my app token. |
Can this be run safely under Linux, or is it bound to Windows? |
I think FFMPEG works better on Linux, but I'm not entirely sure I haven't test it on Linux. And did you modify config.js? |
Yeah, I attached it below with a fake token
…On December 14, 2017 4:18:20 PM MST, Malexion ***@***.***> wrote:
I think FFMPEG works better on Linux, but I'm not entirely sure I
haven't test it on Linux.
And did you modify config.js?
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
#1 (comment)
--
Sent from my Android device with K-9 Mail. Please excuse my brevity.
|
config.js holds the default config settings any modifications you want to make to the config should be passed through main.js in that object that's inserted into the class. It recursively updates the underlying config so you only have to define the properties you want to update. |
Would you elaborate please?
…On December 14, 2017 4:20:49 PM MST, Malexion ***@***.***> wrote:
config.js holds the default config settings any modifications you want
to make to the config should be passed through main.js in that object
that's inserted into the class.
It recursively updates the underlying config so you only have to define
the properties you want to update.
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
#1 (comment)
--
Sent from my Android device with K-9 Mail. Please excuse my brevity.
|
The config is smart enough to only update properties listed. so auto.deafen would be updated if un-commented but the other properties in the auto category would be left alone. |
I see...
That's a bit unintuitive.
A section in the Readme would be beneficial I think.
--
Sent from my Android device with K-9 Mail. Please excuse my brevity.
|
I'm going to try running this under Ubuntu 17.04 and see what happens. This time, I'll put the token in the right spot XP Out of curiosity, is this the source for THE Rhythm Bot? |
well, it worked. Kinda.
|
Scratch the last comment, I was being an idiot and I pasted a web address into the token field. Perhaps I should get some sleep XP |
Hmm still seems to be related to the login process, hard to say what it is. Oh I had to google to figure out what you meant by "THE Rhythm Bot" wow I never knew that existed, I just made this clunky thing about a year ago since I couldn't find any operational music bots at the time. But it looks like https://rythmbot.co/ is just my bot on steroids with sharding to allow easy creation, you could probably rework mine to be just like that if you wanted though it'd take some time. |
Hi, I'm trying to get this up and running,
I'm on Windows 10 with Python 2.7.14, Node.js LTS, and the Latest FFMPEG
I've made the token, and put it into the config, and I get the following error on launch;
What advice do you have?
The text was updated successfully, but these errors were encountered: