-
Notifications
You must be signed in to change notification settings - Fork 341
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Mod action history #2437
Mod action history #2437
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems good, but it looks like I have some back-end fixes to make, related to filtering. Let me work those out fully on my local machine, before merging this.
package.json
Outdated
"engineStrict": true, | ||
"importSort": { | ||
".js, .jsx, .ts, .tsx": { | ||
"style": "module", | ||
"parser": "typescript" | ||
} | ||
} | ||
}, | ||
"packageManager": "pnpm@9.0.5" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is fine, but also I've found using corepack use pnpm@latest
will automatically add the sha for you.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It has the sha now
- Previously when given a post_id, it didn't filter out any other modlog entries, such as community removals. This fixes that problem. - Context: LemmyNet/lemmy-ui#2437
Sweet. I've got the back-end fix now also. |
- Previously when given a post_id, it didn't filter out any other modlog entries, such as community removals. This fixes that problem. - Context: LemmyNet/lemmy-ui#2437 Co-authored-by: SleeplessOne1917 <28871516+SleeplessOne1917@users.noreply.github.com>
Adds frontend for the backend change.