Skip to content
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.

Sign up for GitHub

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

Jump to bottom

DIH4JDA, Sentry & Major Cleanup #293

Merged
merged 209 commits into from
Jul 25, 2022
Merged

DIH4JDA, Sentry & Major Cleanup #293

merged 209 commits into from
Jul 25, 2022

Conversation

jasonlessenich
Copy link
Member

@jasonlessenich jasonlessenich commented May 27, 2022 •
edited
Loading

This PR attempts to integrate DIH4JDA into the bot, which makes it easier to manage and create commands. This goes along with a major cleanup of some commands.

Additions

Tested Commands

  • /avatar
  • /redeploy
  • /leaderboard qotw
  • /leaderboard help-experience
  • /leaderboard thanks
  • /regex
  • /help-guidelines
  • /id-calc
  • /poll
  • /embed create
  • /embed edit
  • /warn discard-by-id
  • /warn add
  • /warn discard-all
  • /change-my-mind
  • /profile
  • /db-admin message-cache info
  • /db-admin migrate
  • /db-admin migrations-list
  • /db-admin export-schema
  • /db-admin quick-migrate
  • /format-code
  • /search-web
  • /qotw submissions mark-best
  • /qotw account increment
  • /qotw account set
  • /qotw questions-queue add
  • /qotw questions-queue list
  • /qotw questions-queue remove
  • /timeout add
  • /timeout remove
  • /config set
  • /config list
  • /config get
  • /uptime
  • /serverlock-admin status
  • /say
  • /help account
  • /ping
  • /warns
  • /serverinfo
  • /botinfo
  • /prune
  • /unban
  • /purge
  • /ban
  • /kick
  • /report
  • Search the Web
  • Report Message
  • Format Code
  • Report User
  • Show Warns

Tested Systems

  • Custom Tags
    • /tags-admin create
    • /tags-admin delete
    • /tags-admin edit
    • /tags
    • /tag-view
  • Server Lock
  • Self Roles
    • /self-role status
    • /self-role create
  • Help Channels
    • /unreserve
  • Github Links
  • Message Links
  • Pingable Names
  • Shareknowledge Votes
  • Job Channel Votes
  • Metrics
  • Presence Updater
  • Suggestions
    • /suggestion accept
    • /suggestion decline
    • /suggestion clear
    • /suggestion on-hold
  • Message Cache
  • Starboard
  • AutoMod

@jasonlessenich jasonlessenich self-assigned this May 27, 2022
@jasonlessenich jasonlessenich changed the title DIH4JDA & Major Cleanup DIH4JDA, Sentry & Major Cleanup May 29, 2022
@jasonlessenich
Added /server-lock status and refactored all other moderation commands
d69011d
@jasonlessenich
Merge remote-tracking branch 'origin/main' into dynxsty/dih4jda
55f2669 
# Conflicts:
#	src/main/java/net/javadiscord/javabot/systems/commands/ChangeMyMindCommand.java
#	src/main/java/net/javadiscord/javabot/systems/help/commands/HelpPingCommand.java
#	src/main/java/net/javadiscord/javabot/systems/help/commands/subcommands/HelpAccountSubcommand.java
#	src/main/java/net/javadiscord/javabot/util/InteractionUtils.java
@jasonlessenich
Upgraded JDA & DIH4JDA and added CommandPermissions
2d9a839
@jasonlessenich
Further implemented DIH4JDA
a2666fd
@jasonlessenich
Merge branch 'main' into dynxsty/dih4jda
2ac3e07 
# Conflicts:
#	src/main/java/net/javadiscord/javabot/Constants.java
#	src/main/java/net/javadiscord/javabot/tasks/PresenceUpdater.java
@jasonlessenich
Refactored /db-admin commands and upgraded DIH4JDA
43f4536
@jasonlessenich
Refactored /warn commands & /warns
25c005b
@jasonlessenich
Added guild checks
11b46e6
@jasonlessenich
Refactored /suggestion commands
188af5b
@jasonlessenich
Refactored /say & /redeploy and moved self_roles out of the `st…
3149acd 
…aff` package
@jasonlessenich
Started work on refactoring /embeds
e43d477
@jasonlessenich
Re-implemented Custom Commands
bc2c542
@jasonlessenich
Added missing annotations
ecb19c6
@jasonlessenich
Removed Guild-based Slash Command Colors & refactored some more commands
db348ec
@jasonlessenich
Refactored even more commands
8f16349
@jasonlessenich
Introduction of ExceptionLogger#capture
25615fa
@jasonlessenich
Added ExceptionLogger#capture to most catch phrases
0aaa9f9
danthe1st
danthe1st reviewed Jul 25, 2022
View reviewed changes
Copy link
Member

@danthe1st danthe1st left a comment

Choose a reason for hiding this comment

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

When setting the server lock status to the "current" value, it reports being successful. We might want to add an error for that.

@jasonlessenich
Copy link
Member Author

When setting the server lock status to the "current" value, it reports being successful. We might want to add an error for that.

image

danthe1st
danthe1st approved these changes Jul 25, 2022
View reviewed changes
Copy link
Member

@danthe1st danthe1st left a comment

Choose a reason for hiding this comment

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

Whenever you plan to merge and deploy this PR, make sure you are available for a couple of hours so you can revert it if anything bad happens.

@danthe1st danthe1st removed review requested needs to be tested For code that needs to be tested before it can be merged to the production branch. labels Jul 25, 2022
@jasonlessenich jasonlessenich merged commit 8217362 into main Jul 25, 2022
@jasonlessenich jasonlessenich deleted the dynxsty/dih4jda branch July 25, 2022 20:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MoonTM-GIT MoonTM-GIT MoonTM-GIT left review comments

@danthe1st danthe1st danthe1st approved these changes

@liquiddevelopmentnet liquiddevelopmentnet liquiddevelopmentnet approved these changes

Assignees

@jasonlessenich jasonlessenich

Labels
enhancement Improvement of an already existing feature
Projects
None yet
Milestone
No milestone
4 participants
@jasonlessenich @danthe1st @MoonTM-GIT @liquiddevelopmentnet