Fix /wordle Bug, Where for Wins on 5th Try, the Title was 'Lose' | Re…

…move Excess Logging Merge pull request #231 from bob-el-bot/quick-updates
bob-el-bot · Dec 19, 2024 · 9983f81 · 9983f81
2 parents ed6d29c + 500a685
commit 9983f81
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 5 deletions.
diff --git a/commands/wordle-group/helpers/wordleGame.cs b/commands/wordle-group/helpers/wordleGame.cs
@@ -41,7 +41,6 @@ public override async Task StartBotGame(SocketInteraction interaction)
 
             // Get a word
             Word = WordleMethods.GetRandomWord();
-            Console.WriteLine(Word);
 
             await Message.ModifyAsync(x => { x.Content = null; x.Embed = WordleMethods.CreateEmbed(this); });
         }

diff --git a/commands/wordle-group/helpers/wordleMethods.cs b/commands/wordle-group/helpers/wordleMethods.cs
@@ -146,7 +146,7 @@ public static string CreateFinalTitle(Wordle game)
                 stringBuilder.AppendLine($"### 💡 {game.Player1.Mention}'s Wordle Expired!");
                 stringBuilder.AppendLine($"**Answer:** {GetFormattedGuess(game.Word)}");
             }
-            else if (game.GuessesLeft > 0)
+            else if (game.Word == game.Guesses.Last().Guess)
             {
                 stringBuilder.AppendLine($"### 💡 {game.Player1.Mention} Guessed the Wordle in: {GuessCount - game.GuessesLeft}!");
                 stringBuilder.AppendLine(GetCongrats());

diff --git a/general-helpers/time/timestamps.cs b/general-helpers/time/timestamps.cs
@@ -85,9 +85,6 @@ public static string FromDateTime(DateTime dateTime, Formats format, Timezone? t
             // Create DateTimeOffset with the correct offset for the timezone
             var dateTimeOffset = new DateTimeOffset(adjustedDateTime, timeZoneInfo.GetUtcOffset(adjustedDateTime));
 
-            // Log the DateTimeOffset after conversion
-            Console.WriteLine("FromDateTime() after gone to DateTimeOffset: " + dateTimeOffset);
-
             // Return the formatted timestamp string
             return $"<t:{dateTimeOffset.ToUnixTimeSeconds()}:{(char)format}>";
         }