Added /audit button to reports (moderation) by modwodmm · Pull Request #1499 · Together-Java/TJ-Bot

modwodmm · 2026-06-10T17:15:41Z

Adds a button that lets the moderator that clicked it see the audit log of the reported user.
Resolves#1498

Zabuzard · 2026-06-11T06:10:12Z

Please add screenshots of the feature and all edge cases in action, makes reviewing it muuuch easier...^^

modwodmm · 2026-06-11T09:23:26Z

Made some changes as you asked.

Zabuzard · 2026-06-11T12:22:19Z

(please dont resolve comments urself, let the author of the comment do it as confirmation that they consider it done, thanks)

done

modwodmm · 2026-06-11T12:34:29Z

sorry I'm kinda new to this pr stuff

modwodmm · 2026-06-15T09:02:34Z

here are the screenshots too

…rationutils

modwodmm · 2026-06-15T11:29:28Z

Zabuzard · 2026-06-15T11:55:16Z

So it shows the last page only now? not sure how i feel about that. the user could have 100 pages and then u show only a single item bc the last page happened to only include one item yet. it should at least show a full page or have buttons i guess. also, shouldnt it be page 1 and not the last? im pretty sure /audit sorts it with the most recent actions shown first, with a full page

modwodmm · 2026-06-15T16:42:48Z

everything works and the pages turn too I forgot to take screenshot 😭 but tested it and it worked just as intended.

surajkumar

Small change but the rest of the PR at my glance looks good. Will be happy to provide the second tick on Zabu's approval

modwodmm · 2026-06-16T11:52:18Z

this time I didn't forget to take screenshots for page turning

modwodmm · 2026-06-16T13:20:53Z

it works I didn't think it would need all the screenshots

modwodmm · 2026-06-17T17:53:37Z

Made the changes and also changed the name to .json.template this time sorry for the earlier mistake.

modwodmm · 2026-06-17T17:54:32Z

and yeah its not the same screenshot every time I take a new one every time I change the code to show that it is tested and works

modwodmm · 2026-06-18T07:03:10Z

Tested it with "1234567890L" id and it didn't crash just gave the message and also made it ephemeral because Christolis asked it to be ephemeral.

modwodmm · 2026-06-18T07:16:06Z

and also if you like some name changes tell me, I'm not good at naming.

modwodmm · 2026-06-18T10:49:12Z

Looked at it thoroughly and changed the names, there might be some that I missed so if any, tell me I'll change them.

(done)

christolis

While testing this I noticed that its behavior is a little different from the /audit command.

More specifically, assuming that a member has more than a page of actions taken against them, while both this new audit button and the original audit command sort the actions by recency, the new audit command inverts the pages if that makes sense.

I'll let the following screenshots tell the story:

Original /audit command

New Audit button

Notice how on the same user, in the new Audit button, the actions are stacked in the first page.

Another thing I have noticed is that when a user turns a page in an ephemeral message after clicking the Audit button, a new embed is send, but perhaps this is a limitation and I am fine with it if that's the case.

Pressing "Next" or "Previous" sends an entirely new message instead of editing that ephemeral message.

christolis · 2026-06-28T16:43:19Z

 import org.togetherjava.tjbot.features.utils.AmbientColors;
 import org.togetherjava.tjbot.features.utils.MessageUtils;

+import java.awt.*;


Star import

christolis · 2026-06-28T16:43:25Z

+import java.awt.*;
 import java.time.Instant;
 import java.time.temporal.ChronoUnit;
+import java.util.*;


Star import

christolis · 2026-06-28T16:43:47Z

-import net.dv8tion.jda.api.entities.Message;
-import net.dv8tion.jda.api.entities.MessageEmbed;
-import net.dv8tion.jda.api.entities.Role;
+import net.dv8tion.jda.api.entities.*;


Star import

christolis · 2026-06-28T16:50:31Z

+ private static List<List<ActionRecord>> groupActionsByPages(List<ActionRecord> actions) {
+ return ModerationUtils.groupActionsByPages(actions);
 }


What is preventing us from entirely removing this function from this class and directly referencing ModerationUtils.groupActionsByPages() where it's needed? It has one usage anyways, specifically in AuditCommand.java in line 107:
List<List<ActionRecord>> groupedActions = groupActionsByPages(actions);

christolis · 2026-06-28T16:59:25Z

+ * Generates a structural text overview outlining the count total of each action type.
+ *
+ * @param moderationActions a collection of history records
+ * @return a formatted markdown description summary


NIT: The "markdown" word should be capitalized, considering you are referring to the formatting language.

christolis · 2026-06-28T17:05:03Z

+ EmbedBuilder auditEmbed =
+ new EmbedBuilder().setTitle("Audit log of **%s**".formatted(user.getName()))
+ .setAuthor(user.getName(), null, user.getEffectiveAvatarUrl())
+ .setColor(Color.BLACK)


Please make use of the recently introduced AmbientColors class to set the color of this embed.

…mports and some other changes

Button to view reported user's previous warns, mutes and punishments
b4a2d5e

modwodmm requested a review from a team as a code owner June 10, 2026 17:15

Zabuzard changed the title ~~Button to view reported user's previous warns, mutes and punishments~~Added /audit button to reports (moderation)Jun 11, 2026

Zabuzard assigned modwodmm Jun 11, 2026

Zabuzard added enhancement New feature or request priority: normal labels Jun 11, 2026

Zabuzard previously requested changes Jun 11, 2026
View reviewed changes

Changed the code according to the reviewer's requirements
27d66db

surajkumar suggested changes Jun 14, 2026
View reviewed changes

Comment threadapplication/src/main/java/org/togetherjava/tjbot/features/moderation/ReportCommand.java Outdated
Comment threadapplication/src/main/java/org/togetherjava/tjbot/features/moderation/ReportCommand.java Outdated

Changed some stuff as requested
4b9b662

Zabuzard requested changes Jun 15, 2026
View reviewed changes

Comment threadapplication/src/main/java/org/togetherjava/tjbot/features/moderation/audit/AuditCommand.java Outdated

modwodmm requested a review from surajkumar June 15, 2026 10:24

Made the methods private in auditcommand and extracted them into mode…
ece6dbf
…rationutils

Added pages to see throught reported user's history
9c71eca

modwodmm requested a review from Zabuzard June 15, 2026 16:59

surajkumar suggested changes Jun 16, 2026
View reviewed changes

Comment threadapplication/src/main/java/org/togetherjava/tjbot/features/moderation/audit/AuditCommand.java Outdated

Zabuzard requested changes Jun 16, 2026
View reviewed changes

Comment threadapplication/src/main/java/org/togetherjava/tjbot/features/moderation/ReportCommand.java Outdated

Used helper methods so the code is easier to understand
940e390

Zabuzard requested changes Jun 16, 2026
View reviewed changes

Comment threadapplication/src/main/java/org/togetherjava/tjbot/features/moderation/ReportCommand.java Outdated

Used map instead of queue(...queue())
9681b28

Zabuzard requested changes Jun 17, 2026
View reviewed changes

Made some changes as requested also brought back config.json.template
3def321

Removed a comment which suraj asked to be removed
6e1805f

Zabuzard requested changes Jun 18, 2026
View reviewed changes

Made requested changes and made the audit message ephemeral
c919cd9

Zabuzard requested changes Jun 18, 2026
View reviewed changes

Comment threadapplication/src/main/java/org/togetherjava/tjbot/features/moderation/ModerationUtils.java Outdated

Changed the names according to context
7ab5a46

Zabuzard previously requested changes Jun 19, 2026
View reviewed changes

Comment threadapplication/src/main/java/org/togetherjava/tjbot/features/moderation/ReportCommand.java Outdated

Changed unused parameter msg to -
9804c59

modwodmm requested a review from surajkumar June 19, 2026 08:06

surajkumar previously approved these changes Jun 20, 2026
View reviewed changes

modwodmm requested a review from Zabuzard June 20, 2026 17:27

Merge branch 'develop' into develop
5cea235

modwodmm dismissed surajkumar’s stale review via 5cea235June 26, 2026 14:43

Fixed the issues in the CI test
189463b

modwodmm requested a review from surajkumar June 28, 2026 05:08

christolis requested changes Jun 28, 2026
View reviewed changes

The audit button copies the style of the command and also fixed the i…
2e2336d
…mports and some other changes

modwodmm requested a review from christolis June 29, 2026 14:11

Uh oh!

Uh oh!

Conversation

modwodmm commented Jun 10, 2026

Uh oh!

Zabuzard commented Jun 11, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

modwodmm commented Jun 11, 2026

Uh oh!

Zabuzard commented Jun 11, 2026

Uh oh!

modwodmm commented Jun 11, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

modwodmm commented Jun 15, 2026

Uh oh!

Uh oh!

modwodmm commented Jun 15, 2026

Uh oh!

Zabuzard commented Jun 15, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

modwodmm commented Jun 15, 2026

Uh oh!

surajkumar left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

modwodmm commented Jun 16, 2026

Uh oh!

Uh oh!

modwodmm commented Jun 16, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

modwodmm commented Jun 17, 2026

Uh oh!

modwodmm commented Jun 17, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

modwodmm commented Jun 18, 2026

Uh oh!

modwodmm commented Jun 18, 2026

Uh oh!

Uh oh!

modwodmm commented Jun 18, 2026

Uh oh!

Uh oh!

christolis left a comment

Choose a reason for hiding this comment

Uh oh!

christolisJun 28, 2026

Choose a reason for hiding this comment

Uh oh!

christolisJun 28, 2026

Choose a reason for hiding this comment

Uh oh!

christolisJun 28, 2026

Choose a reason for hiding this comment

Uh oh!

christolisJun 28, 2026

modwodmm commented Jun 11, 2026•
edited
Loading

Zabuzard commented Jun 15, 2026•
edited
Loading