-
-
Notifications
You must be signed in to change notification settings - Fork 2
Conversation
…amework 4.6.2. Make the Y axis alphabetical not numerical. Fix bounds for grid co-ordinates. Add credits command.
For installation using mono: |
well we use monodevelop to compile it |
Okay I can confirm that it works with the latest version of mono. |
Issue 1: Can't successfully run/compile using the current hosting solution. (Mono 4.4.2, Ubuntu 17.04) Issue 2: When using a build successfully compiled by @Technochips using Mono 5.0.0, bot immediately crashed due to a missing Issue 3: @Technochips could not successfully run the bot using Mono 5.0.0 (latest mono). He got this when trying to run the bot: |
Hmm. I'll have a look into it. I installed mono a few hours ago so it's definatly at the latest version for me. If you want, I can have a look to see what all the dlls etc. that it's using are and send over versions. When you installed all the nuget packages, were there any errors or warnings? |
at the same time, we are on linux, and that might be no good |
Nahh. I'm fairly sure the point of mono is that it works on both. Are you using a user or bot account to run it on? If you're using a user account, check the email for that account as 400 is the error thrown for "new device". |
well geez, that's a lot of commits |
gamebot/TicTacToe.cs
Outdated
} //example: width is 4, it writes 1234 | ||
for (int i = 0; i < height; i++) | ||
{ | ||
result += '\n'; //everytime it go to the next row in the game, creates a new line | ||
result += "ABCDEFGHIJKLMNOPQRSTUVWXYZ"[i]; //it also enters the current height line | ||
result += ":regional_indicator_" + "abcdefghijklmnopqrstuvwxyz"[i] + ":"; //it also enters the current height line |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this emoji might look invisible on mobile
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay. What's the best way to do the letter emojis then?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
actually i don't know
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay. Do you know how your's is working?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(As in result += (i + 1) + "⃣";
)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
well, it uses emoji 1️⃣ 2️⃣ 3️⃣ 4️⃣ 5️⃣ 6️⃣ 7️⃣ 8️⃣ 9️⃣, which is basically one number next to a keycap character
actually i'm not even sure if it works on mobile 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i looked up on emojipedia, numbers works, but letter doesn't work on apple
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed :P
Reportedly works on .NET core which doesn't support host OS yet (Ubuntu 17.04) - will properly test/merge when support eventually:tm: comes |
Dotnet Core for Ubuntu doesn't contain the required files to run net46 applications, which is what this application is based off of. Therefore, these changes can't currently be used in the current hosting environment. And no, botter, don't go off and port this to python. i dont think mr @Technochips wants to deal with that |
:D I won't. Don't worry. |
Fix #2, #7, #8, #14 and #18
Update to Discord.NET version 1.0.0-rc
Requires .NET framework version > 4.6 (I used 4.6.2). I have not yet tested using mono (because I don't have it) and thus if you're going to pull this, it might be worth putting it in a new branch.
I also apologize for the fact that it's all one large commit. It was many (separated by
.
in the commit message), but because Idownload ZIP
ed instead of forking, I had to fork it and then merge changes manually. Trust me, I tried to get my commits to come across as more than one, but I couldn't work out how.