Skip to content
This repository has been archived by the owner on Oct 2, 2023. It is now read-only.

Mod rewrite #89

Open
wants to merge 145 commits into
base: develop
Choose a base branch
from
Open

Mod rewrite #89

wants to merge 145 commits into from

Conversation

LoCrealloc
Copy link
Member

@LoCrealloc LoCrealloc commented Jul 19, 2021 •
edited
Loading

Description
Rewrite of the mod tools.
Features added:

  • possibility to use other time units than days for mute/ban durations
  • added warn/mute/kick/ban ids to the ulog
  • edit warns/mutes/kicks/bans
  • delete warns/mutes/kicks/bans
  • possibility to add an evidence picture to reports/warns/mute/kicks/bans
  • puts bans initiated through discords ban system into the database
  • mod loop now runs every minute

Added a documentation for the mod tools

Issue
Closes #54
Closes #83
Closes #71

Additional Notes
Database changes required!

LoC added 30 commits June 21, 2021 23:13
Added function to user other units than days for mutes/bans
5e57314
Removed unnecessary variable
dadf5f7
Enhanced code style
5d4df8b
Added whitespaces around operators
5d56820
Made reports available on private messages
8f03625
Added report, warn, kick, mute and ban db ids to ulog for mods
ddf2d4d
Merge branch 'develop' into mod_rewrite
49fa943
Fixed giving too many arguments to pubsub subscription
19ee12a
Added function to attach an evidence image to reports
ce407fe
Added evidence image to database on reports
aba7d8c
Added function to attach an evidence image to warns
e65030a
Added function to attach an evidence image to mutes
79f2cd1
Added function to attach an evidence image to kicks
e4415dd
Added function to attach an evidence image to bans
15c0c2a
Added evidence images to user log
fdd45a0
Added function to edit warn reasons
9615ecd
Removed forgotten print statements
1a01fa0
Added function to edit mute reasons
8298a34
Added function to edit mute durations
199b7f5
Added function to edit kick reasons
c30138c
Added function to edit ban reasons
458d35a
Added a function to edit ban durations
ca83e22
Fixed possibility to use senseless time periods for mute/ban edits
478c50d
Added mod level to database for warns, mutes, kicks, bans
d70eec7
Added setting for whether to send a user a concerning delete message
becc4e0
Merge branch 'develop' into mod_rewrite
a547afb
Added a function to delete warns
288fb8a
Added a function to delete mutes
abad9ad
Added a function to delete kicks
ebbe380
Added a function to delete bans
70dea0c
LoCrealloc added 4 commits May 2, 2022 21:07
@LoCrealloc
Removed code duplicates
3d9a2a7 
Signed-off-by: LoC <[email protected]>
@LoCrealloc
fixed some minor issues
85d89f7 
Signed-off-by: LoC <[email protected]>
@LoCrealloc
Merge branch 'develop' into mod_rewrite
bd33356 
# Conflicts:
#	moderation/mod/cog.py
#	moderation/mod/models.py
#	moderation/mod/translations/en.yml
@LoCrealloc
Merge branch 'develop' into mod_rewrite
ee27096 
# Conflicts:
#	moderation/mod/cog.py
@Tristan-H11
Copy link
Member

Maybe squash all the non-mod-related commits? This PR has way too many files not related to the mod rewrite and thus makes it hard to analyze.
Also this does not need to be such a big PR. You could also split it into individual parts and request them onto the same (temporary) branch and then merge this branch into v3.3.
E.g: One PR for the basic work underlying the new changes, one PR for Bans, one PR for Kicks, and so on.

Sounds like a lot of work, but would be significantly easier to read then 130 files in 180 commits with over 4500 lines :)

@Defelo Defelo changed the base branch from v3.3 to develop June 10, 2022 09:13
LoCrealloc and others added 22 commits June 20, 2022 21:55
@LoCrealloc
Merge branch 'develop' into mod_rewrite
da3ea3a
@LoCrealloc
Fixed breaking changes caused by the library
ab5680a 
Signed-off-by: LoC <[email protected]>
@TheCataliasTNT2k
reviewed code until handle_get_userlog_entries and added todos
0aa6342
@LoCrealloc
Re-added years and months to the duration converter and enhanced its …
9c4d502 
…documentation

Signed-off-by: LoC <[email protected]>
@LoCrealloc
Removed on confirmation handler and replaced it with the class locate…
c8ccd29 
…d in the library

Signed-off-by: LoC <[email protected]>
@LoCrealloc
Fixed bugs in mod loop and ban retriever
6293f60 
Signed-off-by: LoC <[email protected]>
@LoCrealloc
Removed entire mod level comparison functionality again..
4ec42dc 
Signed-off-by: LoC <[email protected]>
@LoCrealloc
Added support for discords timeout feature
6bc8b4a 
Signed-off-by: LoC <[email protected]>
@LoCrealloc
Added support for discords timeout feature
6aec0ff 
Signed-off-by: LoC <[email protected]>
@LoCrealloc
Made the bot do not require a mute role anymore necessarily
987d9ae 
Signed-off-by: LoC <[email protected]>
@LoCrealloc
Added docstrings to functions
d840d87 
Signed-off-by: LoC <[email protected]>
@LoCrealloc
Added pubsub for role updates and made mod cog assign mute role autom…
6d515a4 
…atically when it is set.

also fixed some minor bugs (misspelled variable)

Signed-off-by: LoC <[email protected]>
@LoCrealloc
Made functions private
bc97b64 
Signed-off-by: LoC <[email protected]>
@LoCrealloc
Removed ugly TODO-comments
d9dae40 
Signed-off-by: LoC <[email protected]>
@Defelo
Merge branch 'develop' into mod_rewrite
32a9894
@LoCrealloc
Merge branch 'develop' into mod_rewrite
2489fad
@LoCrealloc
Made content filter userlog query accept parameters that are only nec…
a1f5dd7 
…essary for the mod ulog query

Signed-off-by: LoC <[email protected]>
@LoCrealloc
Merge remote-tracking branch 'origin/mod_rewrite' into mod_rewrite
aff7dd2
@LoCrealloc
Merge branch 'develop' into mod_rewrite
8cb2a8d 
# Conflicts:
#	pubsub.md
@LoCrealloc
Removed duplicated changelog entry
8f28426 
Signed-off-by: LoC <[email protected]>
@LoCrealloc
Fixed wrong translation
58d76cc 
Signed-off-by: LoC <[email protected]>
@LoCrealloc
Added check for the moderate_members permission on mute command
48a83ce 
Signed-off-by: LoC <[email protected]>
@codeclimate
Copy link

codeclimate bot commented Sep 27, 2022

Code Climate has analyzed commit 48a83ce and detected 0 issues on this pull request.

View more on Code Climate.

@Defelo Defelo removed their assignment Oct 2, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Tristan-H11 Tristan-H11 Tristan-H11 left review comments

@github-actions github-actions[bot] github-actions[bot] left review comments

@codeclimate codeclimate[bot] codeclimate[bot] left review comments

@Defelo Defelo Defelo requested changes

@DEL7A DEL7A Awaiting requested review from DEL7A

At least 1 approving review is required to merge this pull request.

Assignees

@LoCrealloc LoCrealloc

Labels
enhancement New feature or request priority: high
Projects
Pull Requests
Status: In Progress
Milestone
No milestone
5 participants
@LoCrealloc @Tristan-H11 @Defelo @DEL7A @TheCataliasTNT2k