Admin Tooling Cherry Picks #1290
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* First part of Remote Bwoinking * This should technically work * No Actoring * Fixes Guid not sending over * Making it work for the final. * Fixes for api * Moar Commants! Just more Comments to mark Frontier changes. * comment * wops * Fixes Naming Rules * I Should also fix the naming in the actually code... * Testing some new code * Naming rule and dependency fix (hopefully) * Serverside Webhook update on external sent ahelp messages * Still get data from custom URL's, even if it dosent match a discord webhook * Apply suggestions from code review (Part 1) Co-authored-by: Whatstone <166147148+whatston3@users.noreply.github.com> * Apply suggestions from code review (Part 2) * Bwoink system suggsetions * missing BwoinkSystem changes * Change access on BwoinkSystem._messageQueues * Updates the Regex to support other Discord Clients (beta, alpha) * Merge Fixes * BwoinkSystem: explicitly match "canary."/"ptb." --------- Co-authored-by: Whatstone <166147148+whatston3@users.noreply.github.com> Co-authored-by: Whatstone <whatston3@gmail.com>
* Make chat rate limits a general-purpose system. Intending to use this with ahelps next. * Rate limt ahelps Fixes #28762 * Review comments
* Adding the admin prefix to the ahelp * Updating the admin prefix * The second update of the admin prefix * Configuration correction
* Clear search criteria on loading aHelp window * Pinning technology. * Relay to aHelp window and discord if a user disconnect/reconnect * Fix pinning localization * Log disconnect, reconnects, bans to relay and admin in aHelp * Drop to 5min to hold active conversations * Update Content.Server/Administration/Systems/BwoinkSystem.cs Co-authored-by: Chief-Engineer <119664036+Chief-Engineer@users.noreply.github.com> * discord text styling if diconnect,reconnect,banned message. * Pin icons instead of text * Better Icons * space * Move button generation in to its own XAML * List entry control * Fix spaces * Remove from active conversations on banned * Discord if else block cleanup * Better pin icons * Move icons to stylesheet styleclass * Better field order. * PR review fixes * fixes --------- Co-authored-by: Chief-Engineer <119664036+Chief-Engineer@users.noreply.github.com> Co-authored-by: metalgearsloth <comedian_vs_clown@hotmail.com>
* Move PlayerRateLimitManager to shared * Add interaction rate limits * uncap tests
* A comprehensive rule list for joining admins and mid round command to get rule list added * Fix up for when a rule is added vs started and some logging * fix command help localization, fix admin flags and spam anouncement. * Send admin message only to the joining player not all admins. * Bit better formatting in chat box
github-actions
bot
added
Changes: C#
|
Test fails unrelated |
FoxxoTrystan
requested review from
a team,
VMSolidus,
Aidenkrz,
FoxxoTrystan,
Remuchi,
DEATHB4DEFEAT,
Peptide90 and
Pspritechologist
and removed request for
a team
FoxxoTrystan
added
Priority: 3-Medium
|
UPDATE: The bot side is ready, so long as you are using a RedBot instance. Cog is located here |
VMSolidus
approved these changes
Nov 29, 2024
|
Code review required for latest commit, made custom changes to allow for discord role being used as a title, discord color being used as color, and allowing you to change the admin bwoink name color. |
OldDanceJacket
approved these changes
Nov 30, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Link to every PR I cherry-picked:
The main purpose of this PR is the first cherry-picked PR. It adds the ability for admins to reply to ahelps via discord using an HTTP POST request. See all relevant details in the initial PR by Myzumi.