Skip to content

Work/audit log - #136

Merged
ashquarky merged 15 commits into
devfrom
work/audit-log
Jun 25, 2025
Merged

Work/audit log#136
ashquarky merged 15 commits into
devfrom
work/audit-log

Conversation

@CaramelKat

Copy link
Copy Markdown
Member

Resolves #102

Changes:

This introduces an audit log for different moderator actions. For now, the following are logged:

  • User profile updates (bans, reason and lift dates for bans)
  • Ignoring reports
  • Removing posts
  • Making a community
  • Updating a community
  • Deleting a community

For now, the 10 most recent actions on a user profile are displayed in the UI. A different PR will be opened later with an actual interface to look at all

@CaramelKat
CaramelKat requested review from ashquarky and mrjvs June 21, 2025 23:48

@mrjvs mrjvs left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a few bits and bops left

Comment thread apps/juxtaposition-ui/src/models/logs.js Outdated
Comment thread apps/juxtaposition-ui/src/services/juxt-web/routes/admin/admin.js Outdated
Comment thread apps/juxtaposition-ui/src/services/juxt-web/routes/admin/admin.js Outdated
Comment thread apps/juxtaposition-ui/src/services/juxt-web/routes/console/posts.js Outdated
@CaramelKat
CaramelKat requested a review from mrjvs June 22, 2025 18:23
@CaramelKat

Copy link
Copy Markdown
Member Author

Review items should be covered now

mrjvs
mrjvs previously approved these changes Jun 22, 2025
@mrjvs

mrjvs commented Jun 22, 2025

Copy link
Copy Markdown
Contributor

Looks good, @ashquarky do you want to still look at it before I merge?

@binaryoverload binaryoverload left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a suggestion for the messages to make them a bit nicer to look at - doesn't make a functional difference so just leaving as a comment

There's multiple instances of this, but I've only left one comment as the example

Comment thread apps/juxtaposition-ui/src/services/juxt-web/routes/admin/admin.js Outdated

@ashquarky ashquarky left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small tweak to the DB schema, otherwise I'm down

Comment thread apps/juxtaposition-ui/src/models/logs.js Outdated
Comment thread apps/juxtaposition-ui/src/models/logs.js Outdated
Comment thread apps/juxtaposition-ui/src/services/juxt-web/routes/admin/admin.js Outdated
Comment thread apps/juxtaposition-ui/src/services/juxt-web/routes/admin/admin.js Outdated
Comment thread apps/juxtaposition-ui/src/services/juxt-web/routes/admin/admin.js Outdated
@CaramelKat

Copy link
Copy Markdown
Member Author

Think this should be in a good spot now, all the review items should be addressed.

@CaramelKat
CaramelKat requested review from ashquarky and mrjvs June 24, 2025 23:12

@ashquarky ashquarky left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ashquarky
ashquarky merged commit b49df5e into dev Jun 25, 2025
13 checks passed
@ashquarky
ashquarky deleted the work/audit-log branch June 25, 2025 05:49
@mrjvs mrjvs mentioned this pull request Jun 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature]: Moderation - Interface for Logged Actions

4 participants