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.

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

Implement guild configuration #146

Merged
merged 58 commits into from
Jul 24, 2024
Merged

Implement guild configuration #146

merged 58 commits into from
Jul 24, 2024

Conversation

No767
Copy link
Member

@No767 No767 commented Jul 2, 2024

Summary

This PR focuses on implementing an configuration system for guilds. The guild's configuration is entirely handled within the database itself. The configurations include these settings:

  • Guild age and account ages
  • Custom mentions for posts
  • Anon replies, anon replies without commands, and other boolean options

This is inspired by ModMail's configuration system, but in an more compact and better way.

Warning

This PR may break Rodhaj as an whole. Most of the features should continue to work, but may have some faulty code that may need to be retested.

Tasks

There are some tasks that need to be completed, namely:

  • Sorting for config options
  • Separate age settings into different commands
  • Add time parsing code (with timedeltas)
  • JSON-based help documentation

Types of changes

What types of changes does your code introduce to Rodhaj
Put an x in the boxes that apply

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Update (Updates to README.md, the documentation, etc)
  • Other (if none of the other choices apply)

Checklist

Put an x in the boxes that apply

  • If code changes were made then they have been tested.
    • I have updated the documentation to reflect the changes. (if appropriate)
  • All workflows (except pre-commit.ci) pass with my new changes
  • This PR does not address a duplicate issue or PR

@No767 No767 linked an issue Jul 3, 2024 that may be closed by this pull request
@No767 No767 self-assigned this Jul 16, 2024
@No767 No767 marked this pull request as ready for review July 17, 2024 03:13
Copy link

sonarcloud bot commented Jul 24, 2024

@No767 No767 merged commit 9760ba2 into main Jul 24, 2024
9 checks passed
@No767 No767 deleted the noelle/guild-configs branch July 24, 2024 15:44
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.

Configuration System
1 participant