[PR #248] [MERGED] Make warnings permanent, only mark them as retracted #326

Closed
opened 2026-02-15 15:55:40 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/RPCS3/discord-bot/pull/248
Author: @13xforever
Created: 3/7/2019
Status: Merged
Merged: 3/7/2019
Merged by: @13xforever

Base: masterHead: vnext


📝 Commits (5)

  • ac5ba50 more notes for log parsing results
  • a1f3bb3 more diagnostics
  • 981bc40 make warnings permanent, log retractions, but leave the records intact
  • ba9a4e5 !warn list recent
  • 0208076 command to revert warning status

📊 Changes

10 files changed (+520 additions, -38 deletions)

View changed files

📝 CompatBot/Commands/Warnings.ListGroup.cs (+42 -16)
📝 CompatBot/Commands/Warnings.cs (+107 -20)
📝 CompatBot/Database/BotDb.cs (+4 -0)
CompatBot/Database/Migrations/BotDb/20190307173026_PermanentWarnings.Designer.cs (+268 -0)
CompatBot/Database/Migrations/BotDb/20190307173026_PermanentWarnings.cs (+50 -0)
📝 CompatBot/Database/Migrations/BotDb/BotDbModelSnapshot.cs (+12 -0)
📝 CompatBot/EventHandlers/LogParsing/LogParser.LogSections.cs (+5 -0)
📝 CompatBot/Utils/ResultFormatters/LogParserResultFormatter.GeneralNotesSection.cs (+6 -0)
📝 CompatBot/Utils/ResultFormatters/LogParserResultFormatter.WeirdSettingsSection.cs (+7 -0)
📝 CompatBot/Utils/StringUtils.cs (+19 -2)

📄 Description

  • warning removal now requires reason
  • warnings could be removed or re-issued with the new !warn revert command
  • some new notes for log parsing

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/RPCS3/discord-bot/pull/248 **Author:** [@13xforever](https://github.com/13xforever) **Created:** 3/7/2019 **Status:** ✅ Merged **Merged:** 3/7/2019 **Merged by:** [@13xforever](https://github.com/13xforever) **Base:** `master` ← **Head:** `vnext` --- ### 📝 Commits (5) - [`ac5ba50`](https://github.com/RPCS3/discord-bot/commit/ac5ba505a5145e8064461c35b48705cbaa9059dd) more notes for log parsing results - [`a1f3bb3`](https://github.com/RPCS3/discord-bot/commit/a1f3bb3736c05c0f19a3c39877b7077676f4cb38) more diagnostics - [`981bc40`](https://github.com/RPCS3/discord-bot/commit/981bc40712b4a24ee9dfa604cb3bea6a31b1f4d7) make warnings permanent, log retractions, but leave the records intact - [`ba9a4e5`](https://github.com/RPCS3/discord-bot/commit/ba9a4e5841d86c2f3c02009d3cf416335b64147e) !warn list recent - [`0208076`](https://github.com/RPCS3/discord-bot/commit/0208076c4f65f0a1331812e823bc73da8aad50a8) command to revert warning status ### 📊 Changes **10 files changed** (+520 additions, -38 deletions) <details> <summary>View changed files</summary> 📝 `CompatBot/Commands/Warnings.ListGroup.cs` (+42 -16) 📝 `CompatBot/Commands/Warnings.cs` (+107 -20) 📝 `CompatBot/Database/BotDb.cs` (+4 -0) ➕ `CompatBot/Database/Migrations/BotDb/20190307173026_PermanentWarnings.Designer.cs` (+268 -0) ➕ `CompatBot/Database/Migrations/BotDb/20190307173026_PermanentWarnings.cs` (+50 -0) 📝 `CompatBot/Database/Migrations/BotDb/BotDbModelSnapshot.cs` (+12 -0) 📝 `CompatBot/EventHandlers/LogParsing/LogParser.LogSections.cs` (+5 -0) 📝 `CompatBot/Utils/ResultFormatters/LogParserResultFormatter.GeneralNotesSection.cs` (+6 -0) 📝 `CompatBot/Utils/ResultFormatters/LogParserResultFormatter.WeirdSettingsSection.cs` (+7 -0) 📝 `CompatBot/Utils/StringUtils.cs` (+19 -2) </details> ### 📄 Description * warning removal now requires reason * warnings could be removed or re-issued with the new `!warn revert` command * some new notes for log parsing --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-15 15:55:40 -05:00
yindo closed this issue 2026-02-15 15:55:40 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: RPCS3/discord-bot#326